[prev in list] [next in list] [prev in thread] [next in thread] 

List:       jakarta-commons-dev
Subject:    Re: svn commit: r789561 - /commons/proper/lang/trunk/src/java/org/apache/commons/lang/text/translate
From:       sebb <sebbaz () gmail ! com>
Date:       2009-06-30 11:29:19
Message-ID: 25aac9fc0906300429q84a7025yc30a0d4fe2c45bcf () mail ! gmail ! com
[Download RAW message or body]

On 30/06/2009, sebb <sebbaz@gmail.com> wrote:
> On 30/06/2009, bayard@apache.org <bayard@apache.org> wrote:
> > Author: bayard
> > Date: Tue Jun 30 05:09:01 2009
> > New Revision: 789561
> > 
> > URL: http://svn.apache.org/viewvc?rev=789561&view=rev
> > Log:
> > Added todo note
> > 
> > Modified:
> > commons/proper/lang/trunk/src/java/org/apache/commons/lang/text/translate/EntityArrays.java
> >  
> > Modified: commons/proper/lang/trunk/src/java/org/apache/commons/lang/text/translate/EntityArrays.java
> >                 
> > URL: http://svn.apache.org/viewvc/commons/proper/lang/trunk/src/java/org/apache/co \
> > mmons/lang/text/translate/EntityArrays.java?rev=789561&r1=789560&r2=789561&view=diff
> >  ==============================================================================
> > --- commons/proper/lang/trunk/src/java/org/apache/commons/lang/text/translate/EntityArrays.java \
> >                 (original)
> > +++ commons/proper/lang/trunk/src/java/org/apache/commons/lang/text/translate/EntityArrays.java \
> > Tue Jun 30 05:09:01 2009 @@ -22,6 +22,7 @@
> > *
> > * @since 3.0
> > */
> > +// TODO: These need to be public - make methods to return them for security \
> > purposes?
> 
> 
> These really ought to be private, with public getter, to prevent
> accidental or malicious changes to the entries.

I see you have done just that in a later commit.
Sorry for the noise.

> 
> > class EntityArrays {
> > 
> > static final String[][] ISO8859_1_ESCAPE = {
> > 
> > 
> > 
> 

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


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic