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 09:52:53 GMT
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
>
>
>
Mime
View raw message