20453 2012-12-20 04:55:45 +0000 3.1: two questions re underscore removal 2013-08-06 04:10:46 +0000 1 1 1 Unclassified WebAppsWG WebIDL unspecified All All CLOSED FIXED [v1] P2 normal --- 1 jmdyck cam mike public-script-coord public-webapps-bugzilla oldest_to_newest 80428 0 jmdyck 2012-12-20 04:55:45 +0000 In 3.1, a couple of places say: the identifier [of some construct] is the value of the identifier token with any single leading U+005F LOW LINE ("_") character (underscore) removed. (1) The significance of the word 'single' is unclear. It could mean: (a) if the value of the identifier token begins with exactly one underscore, it is removed. or (b) if the value of the identifier token begins with one or more underscores, exactly one is removed. E.g., if the value of the identifier token is "__foo", is the identifier (a) '__foo' or (b) '_foo' ? (2) The production: identifier = [A-Z_a-z][0-9A-Z_a-z]* allows an identifier token whose value is a single underscore. This seems to imply that the resulting identifier has no characters. Is that intended? 91593 1 cam 2013-08-04 05:46:59 +0000 I removed the "single"s and updated the regex for the identifier token. http://dev.w3.org/cvsweb/2006/webapi/WebIDL/Overview.xml.diff?r1=1.673;r2=1.674;f=h http://dev.w3.org/cvsweb/2006/webapi/WebIDL/v1.xml.diff?r1=1.115;r2=1.116;f=h 91624 2 jmdyck 2013-08-04 20:24:29 +0000 I've since noticed two things: (1) Both the old and new regex allow an identifier token such as _42, which would lead to an identifier of just 42, which seems undesireable. (2) "The identifier of any of the abovementioned IDL constructs MUST NOT ... begin with a U+005F LOW LINE ("_") character." Which would seem to disallow an identifier token (or at least one that provides the identifier for some IDL construct) from having two or more leading underscores (which would obviate the original question about underscore removal). So I wonder if the regex for identifier token should be: _?[A-Za-z][0-9A-Z_a-z]* 91674 3 cam 2013-08-06 01:32:19 +0000 Sounds good, thanks. http://dev.w3.org/cvsweb/2006/webapi/WebIDL/Overview.xml.diff?r1=1.687;r2=1.688;f=h http://dev.w3.org/cvsweb/2006/webapi/WebIDL/v1.xml.diff?r1=1.129;r2=1.130;f=h