[webkit-dev] TestExpectations syntax changes, last call (for a while, at least) ...

Dirk Pranke dpranke at chromium.org
Wed Jun 13 17:49:49 PDT 2012


On Wed, Jun 13, 2012 at 5:42 PM, Maciej Stachowiak <mjs at apple.com> wrote:
>
> On Jun 13, 2012, at 3:58 PM, Darin Adler <darin at apple.com> wrote:
>
>> On Jun 13, 2012, at 3:53 PM, Dirk Pranke <dpranke at chromium.org> wrote:
>>
>>> * we use "\" (backslash) as a delimiter instead of ":" and "="
>>
>> Seems worse to me. When I see a backslash I assume it’s a line continuation character or a C escape sequence.
>>
>> On the other hand, neither the ":" nor the "=" meant anything to me either, and I suggested dropping the delimiter entirely, so I’m not sure my feedback will get listened to!
>>
>>> * As of now the keywords remain all UPPERCASE. I'm happy to change them to all lowercase or mixed case if there's a consensus that such a change is desired.
>>
>> I DON'T LIKE READING THINGS IN ALL UPPERCASE AND WOULD VERY MUCH LIKE THAT TO CHANGE.
>
> I also hate all-caps keywords and would prefer mixed case or all-lowercase, but I'm willing to discuss that for a future round of changes if we can't get agreement now.
>

I think most of the people who liked all-caps mostly wanted there to
be *some* sort of delimiter.

Anyone not okay with:

webkit.org/12345 [Win Mac Debug]
animations/stop-animation-on-suspend.html [Crash Text Pass]

?

-- Dirk


More information about the webkit-dev mailing list