Descriptionthis fixes issue 1452
http://code.google.com/p/google-caja/issues/detail?id=1452
the method reuse() creates a temporary var for an expression, so that
the value can be used multiple times. When reuse() notices the
expression is simple enough, it doesn't create a temporary.
The method reuseAll() is reuse() applied to an argument list.
Currently it just calls reuse() on each argument, which is wrong.
When an argument list is something like this:
o.m(i, i++)
the result is something like this:
var x1___ = i++;
o.m(i, x1___);
This change ensures reuseAll() creates temporaries for all its args
if any of its args needs a temporary.
Patch Set 1 #
MessagesTotal messages: 3
|
||||||||||||||||||||||||||||