Rename variables declared in single var statement

[Frederico Caldeira Knabben]

Input Code

function Testing()
{
    var test1 = 1, test2 = 2, test3 = 3 ;

    alert( test1 + test2 + test3 ) ;
}

Current Results

function Testing(){var A=1,test2=2,test3=3;alert(A+test2+test3);}

Expected Results

function Testing(){var A=1,B=2,C=3;alert(A+B+C);}

[Frederico Caldeira Knabben]

The proposed patch now do the variable search in two steps. The first identify the var declaration blocks. The second extract all names defined in that block.

[Frederico Caldeira Knabben]

Discussion about it with Martin, we found out that variables with the "$" character are not being matched by the proposed patch.

[Frederico Caldeira Knabben]

The new patch now matches any kind of character in the variable name. It also fixes a bug with special regex characters (like "$") in the variable name.

[Frederico Caldeira Knabben]

Fixed with [2268].

I've tested it over the trunk and I still found some issues that have been addressed in the committed code.