Summary: | Compiler warnings that need to be fixed | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Eleni Maria Stea <estea> | ||||||
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | NEW --- | ||||||||
Severity: | Trivial | CC: | alecflett, beidson, esprehn+autocc, estea, ews-watchlist, glenn, gyuyoung.kim, hi, joepeck, jsbell, macpherson, menard, webkit-bug-importer | ||||||
Priority: | P2 | Keywords: | InRadar | ||||||
Version: | WebKit Local Build | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Linux | ||||||||
Attachments: |
|
Description
Eleni Maria Stea
2021-04-06 03:01:30 PDT
Created attachment 425260 [details]
Proposed patch for 224231
proposed patch, fixes compiler warnings
broke the Apple build, working on v2 Comment on attachment 425260 [details] Proposed patch for 224231 View in context: https://bugs.webkit.org/attachment.cgi?id=425260&action=review > Source/WebCore/Modules/indexeddb/server/SQLiteIDBBackingStore.cpp:1278 > + WTFLogAlways("SQLiteIDBBackingStore::databasesSizeForDirectory filePath='%s', database='%s', size=%l" PRIu64, file.utf8().data(), databaseName.utf8().data(), fileSize); Here the `PRIu64` is the format string value. It is like `%ull`. Perhaps an include is missing for your particular port? Created attachment 427137 [details]
Patch
(In reply to Joseph Pecoraro from comment #3) > Comment on attachment 425260 [details] > Proposed patch for 224231 > > View in context: > https://bugs.webkit.org/attachment.cgi?id=425260&action=review > > > Source/WebCore/Modules/indexeddb/server/SQLiteIDBBackingStore.cpp:1278 > > + WTFLogAlways("SQLiteIDBBackingStore::databasesSizeForDirectory filePath='%s', database='%s', size=%l" PRIu64, file.utf8().data(), databaseName.utf8().data(), fileSize); > > Here the `PRIu64` is the format string value. It is like `%ull`. Perhaps an > include is missing for your particular port? You must be right, I've changed my patch to fix the other two warnings, and I'm going to investigate this further. Thank you! |