Opened 13 years ago
Closed 11 years ago
#8003 closed Bug (wontfix)
CKPackager: issue with leaking variables
| Reported by: | Wiktor Walc | Owned by: | |
|---|---|---|---|
| Priority: | Normal | Milestone: | |
| Component: | Project : CKPackager | Version: | |
| Keywords: | Cc: |
Description
Just to not lose it, #8000 revealed a problem with CKPackager. I'm leaving a ticket to investigate it.
Attachments (1)
Change History (7)
comment:1 Changed 13 years ago by
| Status: | new → confirmed |
|---|
comment:2 Changed 13 years ago by
| Status: | confirmed → pending |
|---|
comment:3 Changed 13 years ago by
| Milestone: | → CKEditor 3.6.2 |
|---|---|
| Status: | pending → confirmed |
Changed 13 years ago by
| Attachment: | 8003_TC.patch added |
|---|
comment:4 Changed 13 years ago by
After a quick check the fixes of this issue requires huge change to the current packager, which currently does a single pass on both parsing and outputting, while detecting duplicate identifiers requires a pre-process of building symbol tree.
Right now we can workaround this issue by avoid using short variable names, while obviously it's not bulletproof.
comment:5 Changed 13 years ago by
| Milestone: | CKEditor 3.6.2 |
|---|
comment:6 Changed 11 years ago by
| Resolution: | → wontfix |
|---|---|
| Status: | confirmed → closed |
Since we use Google Closure Compiler it's no longer an issue that we need to fix.
Note: See
TracTickets for help on using
tickets.
Please add a simple TC here.