Closed
Bug 510472
Opened 16 years ago
Closed 16 years ago
[PATCH] Minor code cleanup
Categories
(Skywriter Graveyard :: General, defect)
Skywriter Graveyard
General
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: olle.jonsson, Assigned: olle.jonsson)
Details
Attachments
(1 file)
User-Agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1.2) Gecko/20090729 Firefox/3.5.2
Build Identifier:
http://bespin.pastebin.com/m1e8c3a39 is a patch against Bespin trunk. It fixes a syntactic boo-boo: a trailing comma in a list. And removes a "case" case written twice in the same list. It also makes parsing the regexes in the file simpler: using parens around the regex.
Reproducible: Always
|
Assignee | |
Comment 1•16 years ago
|
||
|
|
Assignee | |
Updated•16 years ago
|
Summary: removes superfluous JS comma, removes superfluous double case case, parens around regexen → [PATCH] removes superfluous JS comma, removes superfluous double case case, parens around regexen
|
||
Comment 2•16 years ago
|
||
For the record, things like this are better suited for the bespin-core mailing list.
Assignee: nobody → olle.jonsson
Severity: normal → minor
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: [PATCH] removes superfluous JS comma, removes superfluous double case case, parens around regexen → [PATCH] Minor code cleanup
Version: unspecified → Trunk
|
||
Comment 3•16 years ago
|
||
These files (fulljslint.js and jsdefs.js) are from other projects (JSLint and
Narcissus) so I'd rather we avoid keeping private forks of these things... if
changes need to be made, better to make them upstream and pull.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•