Opened 12 years ago

Closed 11 years ago

#1240 closed Bug (fixed)

Nested comments break the compressed output

Reported by: Frederico Caldeira Knabben Owned by: Frederico Caldeira Knabben
Priority: Normal Milestone:
Component: Project : CKPackager Version:
Keywords: Confirmed Review+ Cc: julia.frigiere@…

Description

Nested comments work in JavaScript FCKpackager cannot handle these:

/* asdkfjasdfk
as as;dfkj;lj
asdflkjasdfjk
// */

Attachments (1)

1240.patch (799 bytes) - added by Frederico Caldeira Knabben 11 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 Changed 11 years ago by Wojciech Olchawa

Keywords: Confirmed added

comment:2 Changed 11 years ago by Julia

Cc: julia.frigiere@… added

Changed 11 years ago by Frederico Caldeira Knabben

Attachment: 1240.patch added

comment:3 Changed 11 years ago by Frederico Caldeira Knabben

Keywords: Review? added
Owner: set to Frederico Caldeira Knabben
Status: newassigned

The fix for it is quite simple. It is just a matter of doing the /**/ processing before the // one.

comment:4 Changed 11 years ago by Martin Kou

Keywords: Review+ added; Review? removed

comment:5 Changed 11 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: assignedclosed

Fixed with [2202].

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy