Opened 9 years ago

Closed 9 years ago

#4695 closed Bug (fixed)

CKPackager remove mandatory braces

Reported by: Garry Yao Owned by: Garry Yao
Priority: Must have (possibly next milestone) Milestone:
Component: Project : CKPackager Version:
Keywords: Confirmed Review+ Cc:

Description

Having the following input:

if( typeof foo == 'object' )
{
  for( var i in foo )
    if( foo.hasOwnProperty( i ) )
          bar = foo[ i ];
}
else 
   bar = foo;

Will be compressed into:

if( typeof foo == 'object' )
  for( var i in foo )
    if( foo.hasOwnProperty( i ) )
          bar = foo[ i ];
else 
   bar = foo;

Attachments (1)

4695.patch (1.1 KB) - added by Garry Yao 9 years ago.

Download all attachments as: .zip

Change History (4)

Changed 9 years ago by Garry Yao

Attachment: 4695.patch added

comment:1 Changed 9 years ago by Garry Yao

Keywords: Review? added
Owner: set to Garry Yao
Status: newassigned

comment:2 Changed 9 years ago by Frederico Caldeira Knabben

Keywords: Review+ added; Review? removed

comment:3 Changed 9 years ago by Garry Yao

Resolution: fixed
Status: assignedclosed

Fixed with [4649].

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