[webkit-dev] Removal of trailing whitespace

Fujii Hironori fujii.hironori at gmail.com
Wed Apr 12 19:24:03 PDT 2023

I checked the clang-format result of WTF.
$ find Source/WTF -name '*.h' -o -name '*.cpp' -exec clang-format -i '{}'
Although it doesn't comply with the current WebKit style, it looks good
enough to me.
By adopting clang-format for the project, we can forget most parts of
WebKit style guidelines.
It can reduce the memory footprint for WebKit contributors.

However, inline asm are formatted badly, we should disable it for them.
// clang-format off
// clang-format on

On Thu, Apr 13, 2023 at 10:28 AM Michael Catanzaro via webkit-dev <
webkit-dev at lists.webkit.org> wrote:

> On Thu, Apr 13 2023 at 08:15:00 AM +0900, Tetsuharu Ohzeki via
> webkit-dev <webkit-dev at lists.webkit.org> wrote:
> > To digress a little, why does webkit now use a style checker based on
> > python script instead of clang-format?
> > In today, I feel it's more reasonable to use such a formetter.
> We've tried clang-format in several GNOME projects with not great
> results. I'd recommend uncrustify instead.
> Still, I'm not sure it's a good idea for WebKit. I'm sure we could make
> either tool work, but we'd have to be very lax with any configuration
> we use, or it could get pretty annoying. And the existing style checker
> works decently enough.
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> https://lists.webkit.org/mailman/listinfo/webkit-dev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20230413/f35caa4a/attachment.htm>

More information about the webkit-dev mailing list