Summary: | [PlayStation] Fix build break after r267753 | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Don Olmstead <don.olmstead> | ||||
Component: | Platform | Assignee: | Don Olmstead <don.olmstead> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | alecflett, beidson, darin, ews-watchlist, Hironori.Fujii, jsbell, webkit-bug-importer | ||||
Priority: | P2 | Keywords: | InRadar | ||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Don Olmstead
2020-10-01 13:32:03 PDT
Created attachment 410263 [details]
Patch
Committed r267857: <https://trac.webkit.org/changeset/267857> All reviewed patches have been landed. Closing bug and clearing flags on attachment 410263 [details]. Comment on attachment 410263 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=410263&action=review > Source/WebCore/Modules/indexeddb/server/SQLiteIDBBackingStore.cpp:957 > + for (auto& iter : indexInfoMap) { Just noticed this. There are a few things wrong with this: 1) On the WebKit project we prefer words over abbreviations. "iter" is not a word. 2) The identifier "iter" sounds like short for "iterator" but the iterators are automatically handled by a range-based for loop and so this is not an iterator. In contexts like this typically call this "key/value pair" or "pair", even though it’s not a std::pair. |