[Webkit-unassigned] [Bug 97283] New: consider using something other than "[" and "]" for delimiters in the new TestExpectations syntax

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=97283

           Summary: consider using something other than "[" and "]" for
                    delimiters in the new TestExpectations syntax
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Tools / Tests
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: dpranke at chromium.org
                CC: ap at webkit.org, slewis at apple.com, tony at chromium.org,
                    simon.fraser at apple.com, ossy at webkit.org,
                    ojan at chromium.org, jberlin at webkit.org,
                    noel.gordon at gmail.com, rniwa at webkit.org,
                    apavlov at chromium.org, fpizlo at apple.com


Much as I hate to reopen this kettle of fish, I'm starting to think we should use something other than "[" and "]" as the delimiters in the new syntax. When people read the documentation, they've had that notation beaten into them by EBNF to think that that indicates that they are optional. Given that the enclosed sections are optional, this is pretty confusing.

i.e., 

webkit.org/b/12345 [ Debug ] foo.html [ Failure ]

it's easy to see that in docs and think that you don't need to type in the "[" and "]" on the line. 

Perhaps we should switch to something else? On the other hand, people will get the idea pretty quickly so perhaps it's not worth changing?

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.