Re: svn commit: r1480018 [1/2] - in /xmlgraphics/fop/trunk: ./ src/codegen/unicode/java/org/apache/fop/complexscripts/bidi/ src/codegen/unicode/java/org/apache/fop/text/linebreak/ src/java/org/apache/fop/afp/ src/java/org/apache/fop/complexscripts/bidi/ sr...

[Vincent Hennebert]

On 07/05/13 20:53, vhennebert wrote:
> Author: vhennebert
> Date: Tue May  7 18:53:29 2013
> New Revision: 1480018
> 
> URL: http://svn.apache.org/r1480018
> Log:
> Cosmetics only: removed Checkstyle SuppressWithNearbyCommentFilter and fixed 
> corresponding issues
<snip/>
> Modified: xmlgraphics/fop/trunk/checkstyle-5.5.xml
> URL: 
> http://svn.apache.org/viewvc/xmlgraphics/fop/trunk/checkstyle-5.5.xml?rev=1480018&r1=1480017&r2=1480018&view=diff
> ==============================================================================
> --- xmlgraphics/fop/trunk/checkstyle-5.5.xml (original)
> +++ xmlgraphics/fop/trunk/checkstyle-5.5.xml Tue May  7 18:53:29 2013
> @@ -44,14 +44,6 @@
<snip/>
>      <module name="InnerAssignment">
> -      <property name="tokens" 
> value="ASSIGN,BAND_ASSIGN,BOR_ASSIGN,BSR_ASSIGN,BXOR_ASSIGN,DIV_ASSIGN,MINUS_ASSIGN,MOD_ASSIGN,PLUS_ASSIGN,SL_ASSIGN,SR_ASSIGN,STAR_ASSIGN"/>
> +      <property name="tokens" 
> +        
> value="BAND_ASSIGN,BOR_ASSIGN,BSR_ASSIGN,BXOR_ASSIGN,DIV_ASSIGN,MINUS_ASSIGN,MOD_ASSIGN,PLUS_ASSIGN,SL_ASSIGN,SR_ASSIGN,STAR_ASSIGN"/>
>      </module>

Advertising

I removed the ASSIGN operator from the list as it looked reasonable to
me to have inner assignment at those places that became flagged by this
check.

<snip/>

Vincent