0 Replies Latest reply on Nov 25, 2008 12:05 PM by (Peter_Donovan)

    [svn] 4183: The asc parser really only needs the current token, not an array of all pseudoterminal tokens.

      Revision: 4183
      Author: peted@adobe.com
      Date: 2008-11-25 12:04:59 -0800 (Tue, 25 Nov 2008)

      Log Message:
      -----------
      The asc parser really only needs the current token, not an array of all pseudoterminal tokens.
      Currently, the token id is checked to guarantee that the parser is not ever trying to
      read other than the current token. This is temporary, until the Scanner returns a token object.

      On my machine, this change speeds up HospitalApp by 20 seconds and causes minimal changes to
      compile time for any other cases.
      I expect the cause is garbage collection triggering.

      This is the first in series of changes that should simplify/improve the Scanner/Parser interface
      to have the Scanner return a token, rather than a token id
      (an integer, assumed to be an array index or token class if negative).
      These changes should reduce transitory memory use in the Scanner,
      improve clarity of the code and speed compile up slightly.
      They also eliminate the performance reduction on HospitalApp caused by the earlier changes to Names.java

      QE Notes: None
      Doc Notes: None
      tests: checkintests, tamarin, qa performance, asc-tests

      Modified Paths:
      --------------
      flex/sdk/trunk/modules/asc/src/java/macromedia/asc/parser/Parser.java
      flex/sdk/trunk/modules/asc/src/java/macromedia/asc/parser/Scanner.java
      flex/sdk/trunk/modules/asc/src/java/macromedia/asc/parser/Token.java