Opened 17 years ago

Closed 11 years ago

#1243 closed Bug (fixed)

Variable renaming doesn't work on specific case

Reported by: Frederico Caldeira Knabben Owned by:
Priority: Normal Milestone:
Component: Project : CKPackager Version:
Keywords: Cc:

Description

The following input:

function Test()
{
  for ( var idx = 0 ; idx < 10; idx++ )
  {
    if ( ( idx / 2 ) == x( idx / 2 ) )
       alert( idx ) ;
  }
}

Outputs like this:

function Test(){for (var A=0;A<10;A++){if ((A / 2 ) == x( idx / 2)) alert(A);}}

The "idx" variable is not always replaced by "A".

There are also some spaces that could be safely removed (near /), but this is another (minor) issue.

Change History (2)

comment:1 Changed 16 years ago by Wojciech Olchawa

Keywords: Confirmed added

comment:2 Changed 11 years ago by Piotrek Koszuliński

Resolution: fixed
Status: confirmedclosed

This issue expired since we use Google Closure Compiler.

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy