search

rzel / mini-java

Automatically exported from code.google.com/p/mini-java
0 stars 0 forks source link

Don't use field initializer in AbstractParser #30

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
The running time state of RegexParser shows that there are two
_parserConfig instances. Although _parserConfig is private field, the
subclass still has a different instance of the field?

Use constructors for the initialization. I guess we need to call super()?

Original issue reported on code.google.com by lemontree.cool on 1 Oct 2008 at 3:37

GoogleCodeExporter commented 9 years ago

Original comment by lemontree.cool on 3 Oct 2008 at 10:48

GoogleCodeExporter commented 9 years ago 
the second _parserConfig is the cached _parserConfig in subclass; and we don't 
need
to call super() if it's a default one (without parameter);

Original comment by lemontree.cool on 3 Oct 2008 at 10:48