Re: svn commit: r927903 - in /commons/proper/jexl/trunk/src: main/java/org/apache/commons/jexl2/ main/java/org/apache/commons/jexl2/parser/ test/java/org/apache/commons/jexl2/

[sebb <se...@gmail.com>]

On 26/03/2010, henrib@apache.org <he...@apache.org> wrote:
> Author: henrib
>  Date: Fri Mar 26 15:08:12 2010
>  New Revision: 927903
>
>  URL: http://svn.apache.org/viewvc?rev=927903&view=rev
>  Log:
>  Fix JEXL-100;
>  +
>  +public class ASTIntegerLiteral extends JexlNode implements JexlNode.Literal<Integer> {
>  +    /** The type literal value. */
>  +    private Integer literal;

I think this could/should be final.

>  +    public ASTIntegerLiteral(int id) {
>  +        super(id);
>  +    }
>  +
>  +    public ASTIntegerLiteral(Parser p, int id) {
>  +        super(p, id);
>  +    }
>  +
>  +    public Integer getLiteral() {
>  +        if (literal == null) {
>  +            literal = Integer.valueOf(image);
>  +        }
>  +        return literal;
>  +    }
>  +
>  +    /** Accept the visitor. **/
>  +    public Object jjtAccept(ParserVisitor visitor, Object data) {
>  +        return visitor.visit(this, data);
>  +    }
>  +}
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org