[webkit-dev] style warning of includes/alphabetical sorting in FontPlatformDataChromiumWin.cpp

Ryosuke Niwa rniwa at webkit.org
Mon Aug 13 14:13:00 PDT 2012


On Mon, Aug 13, 2012 at 2:10 PM, Koji Ishii <kojiishi at gmail.com> wrote:

> Thanks for the prompt reply. These 3 <> includes are currently in
> trunk. Can you please confirm:
> 1. Should I fix style errors in the original when I change their
> neighbors (and therefore check-webkit-style complains)?
> 2. The following order passes the style, but it mixes Windows headers
> into WebKit's. Are we ok with this?
>

Yes to both.

- Ryosuke
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20120813/38266857/attachment.html>


More information about the webkit-dev mailing list