commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Simone Tripodi <simonetrip...@apache.org>
Subject Re: svn commit: r1198680 - /commons/proper/ognl/trunk/src/main/javacc/ognl.jj
Date Mon, 07 Nov 2011 10:48:39 GMT
ah, I see - didn't read the content, but just the subject. It gives
you anyway more work...
thanks for taking care,
Simo

http://people.apache.org/~simonetripodi/
http://simonetripodi.livejournal.com/
http://twitter.com/simonetripodi
http://www.99soft.org/



On Mon, Nov 7, 2011 at 11:09 AM, Maurizio Cucchiara
<mcucchiara@apache.org> wrote:
> Every commits have a different message, I thought that this would have
> made the peer revision process easier (though it is a big pain in the
> ..., expecially for the mailboxes :D) .
> Furthermore, a detailed message should avoid conditions like this
> http://markmail.org/message/vvkvkochvjeit3g7  :)
>
>
>
>
> Twitter     :http://www.twitter.com/m_cucchiara
> G+          :https://plus.google.com/107903711540963855921
> Linkedin    :http://www.linkedin.com/in/mauriziocucchiara
>
> Maurizio Cucchiara
>
>
> On 7 November 2011 10:52, Simone Tripodi <simonetripodi@apache.org> wrote:
>>
>> Hola Mau,
>> can I ask you why you commit patches step-by-step? Just a matter of curiosity :)
>> Simo
>>
>> http://people.apache.org/~simonetripodi/
>> http://simonetripodi.livejournal.com/
>> http://twitter.com/simonetripodi
>> http://www.99soft.org/
>>
>>
>>
>> On Mon, Nov 7, 2011 at 10:49 AM,  <mcucchiara@apache.org> wrote:
>> > Author: mcucchiara
>> > Date: Mon Nov  7 09:49:05 2011
>> > New Revision: 1198680
>> >
>> > URL: http://svn.apache.org/viewvc?rev=1198680&view=rev
>> > Log:
>> > OGNL-38 - Variable naming, use StringBuilder instead of StringBuffer. Replace
new String(stringBuffer) with stringBuffer.toString().
>> >
>> > Modified:
>> >    commons/proper/ognl/trunk/src/main/javacc/ognl.jj   (contents, props changed)
>> >
>> > Modified: commons/proper/ognl/trunk/src/main/javacc/ognl.jj
>> > URL: http://svn.apache.org/viewvc/commons/proper/ognl/trunk/src/main/javacc/ognl.jj?rev=1198680&r1=1198679&r2=1198680&view=diff
>> > ==============================================================================
>> > --- commons/proper/ognl/trunk/src/main/javacc/ognl.jj (original)
>> > +++ commons/proper/ognl/trunk/src/main/javacc/ognl.jj Mon Nov  7 09:49:05 2011
>> > @@ -847,8 +847,8 @@ void multiplicativeExpression() : {}
>> >
>> >  // unary (level 1)
>> >  void unaryExpression() : {
>> > -    StringBuffer sb;
>> > -    Token t;
>> > +    StringBuilder stringBuilder;
>> > +    Token token;
>> >     ASTInstanceof ionode;
>> >  }
>> >  {
>> > @@ -945,7 +945,7 @@ void unaryExpression() : {
>> >         navigationChain()
>> >         [
>> >             "instanceof"
>> > -            t = <IDENT>/*@bgen(jjtree) #Instanceof( 1) */
>> > +            token = <IDENT>/*@bgen(jjtree) #Instanceof( 1) */
>> >                          {
>> >                            ASTInstanceof jjtn004 = new ASTInstanceof(JJTINSTANCEOF);
>> >                            boolean jjtc004 = true;
>> > @@ -957,15 +957,15 @@ void unaryExpression() : {
>> >                            jjtree.closeNodeScope(jjtn004,  1);
>> >                            jjtc004 = false;
>> >                          }
>> > -/*@egen*/  { sb = new StringBuffer(t.image); ionode = jjtn004; }/*@bgen(jjtree)*/
>> > +/*@egen*/  { stringBuilder = new StringBuilder(token.image); ionode = jjtn004;
}/*@bgen(jjtree)*/
>> >                          } finally {
>> >                            if (jjtc004) {
>> >                              jjtree.closeNodeScope(jjtn004,  1);
>> >                            }
>> >                          }
>> >  /*@egen*/
>> > -            (   "." t = <IDENT>         { sb.append('.').append(
t.image ); }
>> > -            )*                          { ionode.setTargetType(
new String(sb) ); }
>> > +            (   "." token = <IDENT>         { stringBuilder.append('.').append(
token.image ); }
>> > +            )*                          { ionode.setTargetType(
stringBuilder.toString() ); }
>> >         ]
>> >     )
>> >  }
>> > @@ -1067,7 +1067,7 @@ void navigationChain() : {}
>> >
>> >
>> >  void primaryExpression() : {
>> > -    Token   t;
>> > +    Token   token;
>> >     String  className = null;
>> >  }
>> >  {
>> > @@ -1188,7 +1188,7 @@ void primaryExpression() : {
>> >                                              }
>> >  /*@egen*/
>> >      |
>> > -        LOOKAHEAD(2) "#" t=<IDENT>/*@bgen(jjtree) #VarRef( 0) */
>> > +        LOOKAHEAD(2) "#" token=<IDENT>/*@bgen(jjtree) #VarRef( 0)
*/
>> >                                                 {
>> >                                                   ASTVarRef
jjtn007 = new ASTVarRef(JJTVARREF);
>> >                                                   boolean
jjtc007 = true;
>> > @@ -1200,7 +1200,7 @@ void primaryExpression() : {
>> >                                                   jjtree.closeNodeScope(jjtn007,
 0);
>> >                                                   jjtc007
= false;
>> >                                                 }
>> > -/*@egen*/              { jjtn007.setName( t.image ); }/*@bgen(jjtree)*/
>> > +/*@egen*/              { jjtn007.setName( token.image ); }/*@bgen(jjtree)*/
>> >                                                 } finally
{
>> >                                                   if
(jjtc007) {
>> >                                                    
jjtree.closeNodeScope(jjtn007,  0);
>> > @@ -1333,7 +1333,7 @@ void keyValueExpression() : {}
>> >
>> >  void staticReference() : {
>> >     String className = "java.lang.Math";
>> > -    Token t;
>> > +    Token token;
>> >  }
>> >  {
>> >     className=classReference()
>> > @@ -1341,7 +1341,7 @@ void staticReference() : {
>> >             LOOKAHEAD(2)
>> >             staticMethodCall( className )
>> >          |
>> > -            t=<IDENT>/*@bgen(jjtree) #StaticField( 0) */
>> > +            token=<IDENT>/*@bgen(jjtree) #StaticField( 0) */
>> >                                     {
>> >                                       ASTStaticField jjtn001
= new ASTStaticField(JJTSTATICFIELD);
>> >                                       boolean jjtc001 = true;
>> > @@ -1353,7 +1353,7 @@ void staticReference() : {
>> >                                       jjtree.closeNodeScope(jjtn001,
 0);
>> >                                       jjtc001 = false;
>> >                                     }
>> > -/*@egen*/               { jjtn001.init( className, t.image ); }/*@bgen(jjtree)*/
>> > +/*@egen*/               { jjtn001.init( className, token.image ); }/*@bgen(jjtree)*/
>> >                                     } finally {
>> >                                       if (jjtc001) {
>> >                                         jjtree.closeNodeScope(jjtn001,
 0);
>> > @@ -1371,13 +1371,13 @@ String classReference(): {
>> >  }
>> >
>> >  String className(): {
>> > -    Token t;
>> > -    StringBuffer result;
>> > +    Token token;
>> > +    StringBuilder result;
>> >  }
>> >  {
>> > -    t=<IDENT>               { result = new StringBuffer( t.image
); }
>> > -    ( "." t=<IDENT>         { result.append('.').append( t.image
); }
>> > -    )*                      { return new String(result); }
>> > +    token=<IDENT>               { result = new StringBuilder(
token.image ); }
>> > +    ( "." token=<IDENT>         { result.append('.').append( token.image
); }
>> > +    )*                      { return result.toString(); }
>> >  }
>> >
>> >  void constructorCall()       : {/*@bgen(jjtree) Ctor */
>> > @@ -1386,8 +1386,8 @@ void constructorCall()       : {/*@bgen(
>> >     jjtree.openNodeScope(jjtn000);
>> >  /*@egen*/
>> >     String className;
>> > -    Token t;
>> > -    StringBuffer sb;
>> > +    Token token;
>> > +    StringBuilder stringBuilder;
>> >  }
>> >  {/*@bgen(jjtree) Ctor */
>> >     try {
>> > @@ -1486,12 +1486,12 @@ void propertyName()           : {/*@bgen
>> >     boolean jjtc000 = true;
>> >     jjtree.openNodeScope(jjtn000);
>> >  /*@egen*/
>> > -    Token t;
>> > +    Token token;
>> >  }
>> >  {/*@bgen(jjtree) Property */
>> >     try {
>> >  /*@egen*/
>> > -    t=<IDENT>/*@bgen(jjtree) Const */
>> > +    token=<IDENT>/*@bgen(jjtree) Const */
>> >               {
>> >                 ASTConst jjtn001 = new ASTConst(JJTCONST);
>> >                 boolean jjtc001 = true;
>> > @@ -1503,7 +1503,7 @@ void propertyName()           : {/*@bgen
>> >                 jjtree.closeNodeScope(jjtn001, true);
>> >                 jjtc001 = false;
>> >               }
>> > -/*@egen*/ { jjtn001.setValue( t.image ); }/*@bgen(jjtree)*/
>> > +/*@egen*/ { jjtn001.setValue( token.image ); }/*@bgen(jjtree)*/
>> >               } finally {
>> >                 if (jjtc001) {
>> >                   jjtree.closeNodeScope(jjtn001, true);
>> > @@ -1523,18 +1523,18 @@ void staticMethodCall( String className
>> >     boolean jjtc000 = true;
>> >     jjtree.openNodeScope(jjtn000);
>> >  /*@egen*/
>> > -    Token t;
>> > +    Token token;
>> >  }
>> >  {/*@bgen(jjtree) StaticMethod */
>> >     try {
>> >  /*@egen*/
>> > -    t=<IDENT> "(" [ assignmentExpression() ( "," assignmentExpression()
)* ] ")"/*@bgen(jjtree)*/
>> > +    token=<IDENT> "(" [ assignmentExpression() ( "," assignmentExpression()
)* ] ")"/*@bgen(jjtree)*/
>> >                                         {
>> >                                           jjtree.closeNodeScope(jjtn000,
true);
>> >                                           jjtc000 = false;
>> >                                         }
>> >  /*@egen*/
>> > -                                        { jjtn000.init(
className, t.image ); }/*@bgen(jjtree)*/
>> > +                                        { jjtn000.init(
className, token.image ); }/*@bgen(jjtree)*/
>> >     } catch (Throwable jjte000) {
>> >       if (jjtc000) {
>> >         jjtree.clearNodeScope(jjtn000);
>> > @@ -1562,18 +1562,18 @@ void methodCall()         : {/*@bgen(jjt
>> >     boolean jjtc000 = true;
>> >     jjtree.openNodeScope(jjtn000);
>> >  /*@egen*/
>> > -    Token t;
>> > +    Token token;
>> >  }
>> >  {/*@bgen(jjtree) Method */
>> >     try {
>> >  /*@egen*/
>> > -    t=<IDENT> "(" [ assignmentExpression() ( "," assignmentExpression()
)* ] ")"/*@bgen(jjtree)*/
>> > +    token=<IDENT> "(" [ assignmentExpression() ( "," assignmentExpression()
)* ] ")"/*@bgen(jjtree)*/
>> >                                         {
>> >                                           jjtree.closeNodeScope(jjtn000,
true);
>> >                                           jjtc000 = false;
>> >                                         }
>> >  /*@egen*/
>> > -                                        { jjtn000.setMethodName(
t.image ); }/*@bgen(jjtree)*/
>> > +                                        { jjtn000.setMethodName(
token.image ); }/*@bgen(jjtree)*/
>> >     } catch (Throwable jjte000) {
>> >       if (jjtc000) {
>> >         jjtree.clearNodeScope(jjtn000);
>> > @@ -1820,7 +1820,7 @@ TOKEN_MGR_DECLS:
>> >       /** Holds char literal start token. */
>> >     private char charLiteralStartQuote;
>> >       /** Holds the last string literal parsed. */
>> > -    private StringBuffer stringBuffer;
>> > +    private StringBuilder stringBuilder;
>> >
>> >       /** Converts an escape sequence into a character value. */
>> >     private char escapeChar()
>> > @@ -1960,9 +1960,9 @@ MORE:
>> >  {
>> >     "`"     : WithinBackCharLiteral
>> >  |
>> > -    "'"     { stringBuffer = new StringBuffer(); }: WithinCharLiteral
>> > +    "'"     { stringBuilder = new StringBuilder(); }: WithinCharLiteral
>> >  |
>> > -    "\""    { stringBuffer = new StringBuffer(); }: WithinStringLiteral
>> > +    "\""    { stringBuilder = new StringBuilder(); }: WithinStringLiteral
>> >  }
>> >
>> >  <WithinCharLiteral> MORE:
>> > @@ -1971,20 +1971,20 @@ MORE:
>> >                 | (["0"-"3"])? ["0"-"7"] (["0"-"7"])?
>> >                 )
>> >     >
>> > -        { charValue = escapeChar(); stringBuffer.append(charValue); }
>> > +        { charValue = escapeChar(); stringBuilder.append(charValue); }
>> >  |
>> >     < (~["'","\\"]) >
>> > -        { charValue = image.charAt( image.length()-1 ); stringBuffer.append(charValue);
}
>> > +        { charValue = image.charAt( image.length()-1 ); stringBuilder.append(charValue);
}
>> >  }
>> >
>> >  <WithinCharLiteral> TOKEN:
>> >  {
>> >     < CHAR_LITERAL: "'">
>> >         {
>> > -            if (stringBuffer.length() == 1) {
>> > +            if (stringBuilder.length() == 1) {
>> >                 literalValue = new Character( charValue );
>> >             } else {
>> > -                literalValue = new String( stringBuffer );
>> > +                literalValue = stringBuilder.toString();
>> >             }
>> >         }
>> >         : DEFAULT
>> > @@ -2008,16 +2008,16 @@ MORE:
>> >  <WithinStringLiteral> MORE:
>> >  {
>> >     < STRING_ESC: <ESC> >
>> > -        { stringBuffer.append( escapeChar() ); }
>> > +        { stringBuilder.append( escapeChar() ); }
>> >  |
>> >     < (~["\"","\\"]) >
>> > -        { stringBuffer.append( image.charAt(image.length()-1) ); }
>> > +        { stringBuilder.append( image.charAt(image.length()-1) ); }
>> >  }
>> >
>> >  <WithinStringLiteral> TOKEN:
>> >  {
>> >     <STRING_LITERAL: "\"">
>> > -        { literalValue = new String( stringBuffer ); }
>> > +        { literalValue = stringBuilder.toString(); }
>> >         : DEFAULT
>> >  }
>> >
>> >
>> > Propchange: commons/proper/ognl/trunk/src/main/javacc/ognl.jj
>> > ------------------------------------------------------------------------------
>> >    svn:eol-style = native
>> >
>> > Propchange: commons/proper/ognl/trunk/src/main/javacc/ognl.jj
>> > ------------------------------------------------------------------------------
>> >    svn:keywords = Id
>> >
>> >
>> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org
>
>
Mime
View raw message