Re: [webkit-dev] Encoding and decoding ProcessID

2019-02-27 Thread Alex Christensen
WKProcessID is only used in 2 functions that are only used for testing, so it’s probably not too important as long as your tests work. Also, earlier you said "Our uint32_t is a signed integer.” If that’s true, you’re going to have some bigger problems. > On Feb 26, 2019, at 11:47 PM,

[webkit-dev] Reminder regarding formatting uint64_t

2019-02-27 Thread Michael Catanzaro
Hi, For the past several years, I've been regularly fixing -Wformat warnings that look like this: ../../Source/WebKit/WebProcess/WebPage/WebPage.cpp:3148:46: warning: format ‘%llu’ expects argument of type ‘long long unsigned int’, but argument 6 has type ‘uint64_t’ {aka ‘long unsigned

Re: [webkit-dev] Reminder regarding formatting uint64_t

2019-02-27 Thread Ryosuke Niwa
We should probably stop using these formatting strings in favor of makeString by making *LOG* functions to use makeString behind the scene. See https://trac.webkit.org/changeset/242014 for example. - R. Niwa On Wed, Feb 27, 2019 at 2:36 PM Michael Catanzaro wrote: > Hi, > > For the past

Re: [webkit-dev] Reminder regarding formatting uint64_t

2019-02-27 Thread Simon Fraser
Or use LOG_WITH_STREAM() where I presume << pageID() << does the right thing. We’d need some new RELEASE_LOG_ macros for that. Simon > On Feb 27, 2019, at 2:47 PM, Ryosuke Niwa wrote: > > We should probably stop using these formatting strings in favor of makeString > by making *LOG*

Re: [webkit-dev] Reminder regarding formatting uint64_t

2019-02-27 Thread Adrian Perez de Castro
Hello, On a related note... On Wed, 27 Feb 2019 16:35:39 -0600, Michael Catanzaro wrote: > For the past several years, I've been regularly fixing -Wformat > warnings that look like this: > > ../../Source/WebKit/WebProcess/WebPage/WebPage.cpp:3148:46: warning: > format ‘%llu’ expects

Re: [webkit-dev] Reminder regarding formatting uint64_t

2019-02-27 Thread Keith Rollin
> On Wed, Feb 27, 2019 at 2:36 PM Michael Catanzaro > wrote: > Hi, > > For the past several years, I've been regularly fixing -Wformat > warnings that look like this: > > ../../Source/WebKit/WebProcess/WebPage/WebPage.cpp:3148:46: warning: > format ‘%llu’ expects argument of type ‘long long

Re: [webkit-dev] Reminder regarding formatting uint64_t

2019-02-27 Thread Michael Catanzaro
On Wed, Feb 27, 2019 at 6:05 PM, Keith Rollin wrote: The underlying Cocoa os_log facility Yeah this is really interesting too. RELEASE_LOG is Cocoa-specific, because it's only backed by os_log. So when you change debug LOGs to RELEASE_LOG, you're removing the logging for other platforms. I