Opened 14 years ago
Closed 9 years ago
#2429 closed defect (wontfix)
intern-strings should ignore code in comments
Reported by: | James Burke | Owned by: | James Burke |
---|---|---|---|
Priority: | low | Milestone: | future |
Component: | BuildSystem | Version: | 0.4.1 |
Keywords: | Cc: | Rawld Gill | |
Blocked By: | Blocking: |
Description (last modified by )
Right now, intern-strings will try to intern resources even if they are in a comment, like:
//templateCssPath: dojo.uri.dojoModuleUri("dojo.widget", "templates/FooButton.css"),
Change History (8)
comment:1 Changed 14 years ago by
Component: | General → BuildSystem |
---|---|
Owner: | changed from anonymous to James Burke |
comment:2 Changed 14 years ago by
Milestone: | 0.9 → 1.0 |
---|
comment:3 Changed 14 years ago by
Milestone: | 1.0 → 2.0 |
---|
comment:5 Changed 13 years ago by
Description: | modified (diff) |
---|---|
Milestone: | 1.3 → future |
comment:6 Changed 9 years ago by
Cc: | rawld added |
---|---|
Keywords: | needsreview added |
Priority: | high → low |
with the dojo/text plugin I believe this is no longer a concern
comment:7 Changed 9 years ago by
Cc: | Rawld Gill added; rawld removed |
---|
comment:8 Changed 9 years ago by
Keywords: | needsreview removed |
---|---|
Resolution: | → wontfix |
Status: | new → closed |
I agree, this should be a wontfix as the whole dojo.cache() thing isn't being used anymore.
Note: See
TracTickets for help on using
tickets.
Moving to 1.0, not the optimal behavior, but if you use a shrink-safed file or the comment-stripped file (optimize=shrinksafe or optimize=comments for the 0.9 build), the commented out stuff goes away.