commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sebb <seb...@gmail.com>
Subject [JEXL] New non-private mutable fields
Date Thu, 01 Dec 2011 10:45:02 GMT
Clirr reports that there are some new mutable non-private fields.

Mutable fields should be private, and only updated through a setter method.

Can the following be made private?
They don't appear to be used externally.

INFO: 6000: org.apache.commons.jexl2.Interpreter: Added protected
field cancelled

Would need to add a setter for cancelled.

INFO: 6000: org.apache.commons.jexl2.Interpreter: Added protected
field parameters

parameters is currently write-only - it is not read as far as I can
tell - so is it even needed?

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


Mime
View raw message