RESOLVED WONTFIX Bug 97283
consider using something other than "[" and "]" for delimiters in the new TestExpectations syntax 
https://bugs.webkit.org/show_bug.cgi?id=97283
Summary consider using something other than "[" and "]" for delimiters in the new Tes...
Dirk Pranke
Reported 2012-09-20 19:20:53 PDT
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?
Attachments
Add attachment proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1 2012-09-20 19:30:03 PDT
I'm fine with using () or {}. Avoid <> to be easily mixed with HTML. But I don't think people will mistake [] as BNF notation.
Dirk Pranke
Comment 2 2012-09-20 19:39:35 PDT
(In reply to comment #1) > I don't think people will mistake [] as BNF notation. People already have :)
Dirk Pranke
Comment 3 2012-09-24 14:27:27 PDT
Given that no one has chimed in on this besides rniwa, who seemed to think that [] was fine, I'm going to let this dog sleep.
Note You need to log in before you can comment on or make changes to this bug.