[Webkit-unassigned] [Bug 237089] Bubblewrap launcher doesn't bind font locations from XDG_DATA_DIRS

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Feb 23 14:42:53 PST 2022


https://bugs.webkit.org/show_bug.cgi?id=237089

--- Comment #6 from apteryx <maxim.cournoyer at gmail.com> ---
(In reply to Michael Catanzaro from comment #3)
> Comment on attachment 453016 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=453016&action=review
> 
> I agree, except for code style nits:
> 
> > Source/WebKit/UIProcess/Launcher/glib/BubblewrapLauncher.cpp:308
> > +    for (auto dataDir = dataDirs; dataDir != nullptr && *dataDir != nullptr; dataDir++) {
> 
> I'm surprised the style bot didn't complain about this. Always write out the
> * on auto* to make it more clear that it's a pointer, and never explicitly
> compare against nullptr:
> 
> for (auto* dataDir = dataDirs; dataDir && *dataDir; dataDir++) {

Adjusted. Thanks for the prompt review!

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20220223/0fd548f1/attachment-0001.htm>


More information about the webkit-unassigned mailing list