Approved! 5 years later, someone finally does something about that comment.
On 2011-01-11, at 12:10, André Bargull wrote: > Change bargull-20110111-6BT by barg...@bargull02 on 2011-01-11 17:54:15 > in /home/anba/src/svn/openlaszlo/trunk > for http://svn.openlaszlo.org/openlaszlo/trunk > > Summary: use lookup table for operator precedence in ParseTreePrinter > > New Features: LPP-9642 (use lookup table for operator precedence) > > Technical Reviewer: ptw > QA Reviewer: (pending) > > Details: > Replaced the somewhat expensive operator precedence lookup by array iteration > and string matching with a simpler lookup table. Also added a second method > to retrieve precedence for unambiguous operators to avoid passing the boolean > argument every time. > > Also cleaned up import statements and removed dead code. > > > Tests: > > Files: > M WEB-INF/lps/server/src/org/openlaszlo/sc/SWF9ParseTreePrinter.java > M WEB-INF/lps/server/src/org/openlaszlo/sc/ParseTreePrinter.java > > Changeset: > http://svn.openlaszlo.org/openlaszlo/patches/bargull-20110111-6BT.tar >
