Re: [webkit-dev] Naming scheme for fooIfExists/ensureFoo - Round 2

2024-10-03 Thread Jean-Yves Avenard via webkit-dev
> > On 3 Oct 2024, at 1:48 pm, Ryosuke Niwa via webkit-dev > wrote: > > StyleResolver* styleResolverIfExists(); > StyleResolver& styleResolver(); I myself prefer this one, it’s more concise in its most common use. It’s also the style I’ve personally seen the most around the code I’m involved

Re: [webkit-dev] Initializing member variables

2024-09-19 Thread Jean-Yves Avenard via webkit-dev
> On 20 Sep 2024, at 7:55 AM, Ryosuke Niwa via webkit-dev > wrote: > > > Should we do: > > struct Foo { > int bar = 0; > } > > Or > > struct Foo { > int bar { 0 }; > } > > We do both at the moment. > > - R. Niwa I think `int bar = 0` reads better. I only ever see (and use) { }

Re: [webkit-dev] Proposal: Dropping MSVC support and use clang-cl exclusively on Windows

2024-05-01 Thread Jean-Yves Avenard via webkit-dev
Hi > On 2 May 2024, at 10:07 am, Herb Sutter via webkit-dev > wrote: > > > We’ve had full C++20 including concepts for a couple of years so I wasn’t > sure what problem you were running into… are you encountering bugs in those > features? or you can’t turn on /std:c++20 for some reason? or

Re: [webkit-dev] Approving / Rejects PRs on GitHub when not a reviewer?

2023-11-28 Thread Jean-Yves Avenard via webkit-dev
Hi > On 29 Nov 2023, at 9:44 am, Chris Dumez via webkit-dev > wrote: > > FYI, our official documentation on WebKit.org says: > ``` > Making unofficial reviews before you become a reviewer is encouraged. This is > an excellent way to show your skills. Note that you should n

Re: [webkit-dev] Unsigned to avoid negative values

2023-01-24 Thread Jean-Yves Avenard via webkit-dev
I could have sworn reading a few years ago a white paper discussing signed vs unsigned discussed with Blink coding style showing that using unsigned had a performance impact. Of course, now I can’t find reference to it. But I clearly recall recommendations like you mentioned. Sent from my

Re: [webkit-dev] New status field required for feature flags

2023-01-10 Thread Jean-Yves Avenard via webkit-dev
Hi. What status should be used for: "Enabled by default on most platforms except those we’ve decided not to support or if a local system dependency can’t be met but feature is still ready for general use, but UI to toggle the feature remains to support debugging, A/B testing." That defaultValu

Re: [webkit-dev] Running the GTK port on macOS with Docker

2021-11-22 Thread Jean-Yves Avenard via webkit-dev
Hi How is that effort going ? Having a way to build the GTK port today would have saved me a lot of time. Instead I had to keep uploading patches to bugzilla to find compilation errors. Jean-Yves > On 15 Oct 2021, at 10:33 pm, Philippe Normand wrote: > > On Fri, 2021-10-15 at 10:04 +1100, Jea

Re: [webkit-dev] Running the GTK port on macOS with Docker

2021-10-14 Thread Jean-Yves Avenard via webkit-dev
Hi. Personally, I would love a way to quickly build for GTK on my Mac, as for now I had to rely on pushing to the EWS, wait for compilation errors and fix them as I went. So something running in docker using my local tree would be ideal. Jean-Yves > On 15 Oct 2021, at 1:21 am, Philippe Norman