I noticed the project contains the org.apache.sysml.parser.Token class, which is an old JavaCC class that I believe is now obsolete since antlr is used. The Token class itself is only referenced by org.apache.sysml.parser.ParseException. The org.apache.sysml.parser.DMLParseException (subclass of ParseException) references the currentToken (Token) field of ParseException.
If the currentToken, expectedTokenSequences, and tokenImage fields are removed from ParseException, the only code that requires further removal is the small amount of code in DMLParseException that references currentToken, which would seem to indicate that these fields aren't used and can be removed. If those fields are removed, ParseException has no fields. So, I was wondering the following: (1) Can we remove the Token class and the three fields from ParseException and remove the currentToken code from DMLParseException? (2) Does ParseException have a different meaning and is it used differently than DMLParseException, so they should be kept as separate classes, or can they be combined into a single class? Deron
