[webkit-dev] Moving WebCore/accessibility code into WebCore/platform/

Sam Weinig weinig at apple.com
Tue May 21 10:58:07 PDT 2013


On May 21, 2013, at 10:56 AM, Darin Adler <darin at apple.com> wrote:

> On May 21, 2013, at 10:42 AM, Sam Weinig <weinig at apple.com> wrote:
> 
>> On May 21, 2013, at 8:15 AM, Darin Adler <darin at apple.com> wrote:
>> 
>>>> Of course, we could always add an exception to the style checker
>>> 
>>> The style checker rule is wrong and should be removed. It’s better to use a platform abstraction, but a platform #ifdef is also OK.
>> 
>> This was something we discussed during the contributors meeting.  The premise was that we should aim for having no PLATFORM() #ifdefs in non-Platform and and should instead use other macros that are more descriptive.  I like that rule, but if isn't really possible, we should probably remove the check.
> 
> It’s an interesting principle, but I think too far from reality right now.
> 
> And it leads to crazy things like the Clipboard in the dom directory with per-platform subclasses in the platform directory.
> 
> — Darin

Fair enough. I'll talk to Jessie about removing it, and maybe we can revisit it in the future.

- Sam
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20130521/1ae17f04/attachment.html>


More information about the webkit-dev mailing list