1

Closed

multiple Important comments issues

description

ajaxmin doesn't remove empty line, doesn't remove comments between two 'important' comment
 
e.g. have below in js file:
// test1.js
/*! test1.js this is a test /
/// I am a single line comment
/
I am multi
 
 
  • line
  • comment /
     
    /
    * I am multi
  • line
  • comment /
     
    /
    ! I am multi
     
     
  • line
  • comment /
     
     
     
    var js_test = "test";
     
     
    It is minified to:
    /
    ! test1.js this is a test /
    /// I am a single line comment
    /
    I am multi
     
     
  • line
  • comment /
     
    /
    * I am multi
  • line
  • comment /
     
    /
    ! I am multi
     
     
  • line
  • comment */
    var js_test="test";
Closed Aug 14, 2012 at 2:35 AM by ronlo

comments

ronlo wrote Jun 1, 2011 at 5:37 PM

Excellent find, thanks. I've checked in a fix, changelist #74771; it will release with the next drop, version 4.21.

ronlo wrote Jun 2, 2011 at 12:58 AM

Fixed in released version 4.21