Properly parse IDL arrays.

Properly parse IDL arrays.

Arrays, even DOMString[], are not still properly supported in generators
(methods with them are mostly ignored), that'll be fixed later.

R=podivilov@chromium.org,sra@google.com

Committed: https://code.google.com/p/dart/source/detail?r=8892

[Anton Muhin, 2012-06-19 18:05:48]

WebKit IDL recently started to use ScriptProfile[] type.  I've commented it out
in the roll, but want to bring back.

[sra1, 2012-06-19 20:09:40]

LGTM (fix the regex)

https://chromiumcodereview.appspot.com/10591002/diff/1/lib/dom/scripts/idlpar...
File lib/dom/scripts/idlparser.py (right):

https://chromiumcodereview.appspot.com/10591002/diff/1/lib/dom/scripts/idlpar...
lib/dom/scripts/idlparser.py:287: return re.compile('\[\]+')
I think this recognizes [] []] []]] etc
Make the regex a raw string like the others.

r'(\[\])+'

[sra1, 2012-06-19 20:10:13]

On 2012/06/19 20:09:40, sra1 wrote:
> LGTM (fix the regex)
> 
>
https://chromiumcodereview.appspot.com/10591002/diff/1/lib/dom/scripts/idlpar...
> File lib/dom/scripts/idlparser.py (right):
> 
>
https://chromiumcodereview.appspot.com/10591002/diff/1/lib/dom/scripts/idlpar...
> lib/dom/scripts/idlparser.py:287: return re.compile('\[\]+')
> I think this recognizes [] []] []]] etc
> Make the regex a raw string like the others.
> 
> r'(\[\])+'

Thanks for doing this - it has bugged me for a while.

[Anton Muhin, 2012-06-20 09:45:04]

Thanks a lot for review, Stephen, submitting.

https://chromiumcodereview.appspot.com/10591002/diff/1/lib/dom/scripts/idlpar...
File lib/dom/scripts/idlparser.py (right):

https://chromiumcodereview.appspot.com/10591002/diff/1/lib/dom/scripts/idlpar...
lib/dom/scripts/idlparser.py:287: return re.compile('\[\]+')
Done, thanks for spotting this, getting rusty w/ regexps.

On 2012/06/19 20:09:40, sra1 wrote:
> I think this recognizes [] []] []]] etc
> Make the regex a raw string like the others.
> 
> r'(\[\])+'