systemml-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Deron Eriksson <deroneriks...@gmail.com>
Subject parser.Token class and related exceptions
Date Wed, 16 Dec 2015 01:02:01 GMT
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

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message