Possible efficiency improvement for chart lines in VML

[Andrew.Thomas@…]

In dojo/src/charging/vml/Plotters.js, line 536 (the Line: plotter) there is:

if (i==0){

cmd.push("m"); cmd.push(x+","+y);

}else{

cmd.push("l"); cmd.push(x+","+y);

}

My reading of the VML spec suggests that the following is equally correct, and results in a shorter command set. It works in IE7.

if (i==0){

cmd.push("m");

}else if (i==1){

cmd.push("l");

}else{

cmd.push(x+","+y);

[Andrew.Thomas@…]

Oops - typo. The last else{ should not be there. The cmd.push(x+","+y) at the end should be unconditional.

[Tom Trenka]

Going to close this; porting charting to use dojox.gfx, and that efficiency is abstracted there.