On Fri, Sep 4, 2020 at 2:56 PM Ryosuke Niwa <[email protected]> wrote:
> On Thu, Sep 3, 2020 at 10:11 PM Fujii Hironori <[email protected]> > wrote: > >> >> On Fri, Sep 4, 2020 at 1:31 PM Ryosuke Niwa <[email protected]> wrote: >> >>> Consecutive bit fields must use the same type. >>> >> >> RenderLayer is mixing bool and unsigned in the consecutive bit fields. >> They should use only uint8_t, right? >> >> https://trac.webkit.org/browser/webkit/trunk/Source/WebCore/rendering/RenderLayer.h#L1263 >> > > Under the proposed coding style guide, yes, although I highly doubt the > size of RenderLayer really matters. > > Do you mean your new rule is applicable only for performance critical parts? BTW, I don't like to idea adding a new rule, but keeping old style code. It introduces divergence between the guideline and actual code. Do we really need a new rule that one doesn't necessary have to follow?
_______________________________________________ webkit-dev mailing list [email protected] https://lists.webkit.org/mailman/listinfo/webkit-dev
