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

Adam Barth abarth at webkit.org
Wed Jun 13 18:02:54 PDT 2012


On Wed, Jun 13, 2012 at 5:49 PM, Dirk Pranke <dpranke at chromium.org> wrote:
> 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]
>
> ?

I think you need a /b/ in there if you want the link to be copy/pastable:

webkit.org/b/12345

Adam


P.S., Please feel free to ignore this suggestion, but this is my
favorite shade of purple:

animations/stop-animation-on-suspend.html Crash Text Pass (Win Mac Debug)


More information about the webkit-dev mailing list