maven-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Hervé BOUTEMY <herve.bout...@free.fr>
Subject Re: svn commit: r744389 - in /maven/mercury/trunk/mercury-ant-tasks: ./ src/main/java/org/apache/maven/mercury/ant/tasks/ src/main/resources/org/apache/maven/mercury/ant/tasks/ src/test/java/org/apache/maven/mercury/ant/tasks/ src/test/resources/ src/test/...
Date Sat, 14 Feb 2009 08:12:43 GMT
Oleg,

Sorry to bug you about coding style, but this is important for the whole team 
to be able to maintain code.

For conditional (and loops):
- one space after if/for statement
- always add a block, even if it's not technically necessary

Regards,

Hervé

Le samedi 14 février 2009, ogusakov@apache.org a écrit :
> Author: ogusakov
> Date: Sat Feb 14 04:36:21 2009
> New Revision: 744389
>
> URL: http://svn.apache.org/viewvc?rev=744389&view=rev
> Log:
> [MNG-3969] intermediate commit
>
[SNAP]
> +
> +    @Override
> +    public void setId( String id )
> +    {
> +        super.setId( id );
> +
> +        if( _sourceDependency != null )
> +            _sourceDependency.setId( id );
> +    }
> +
> +    public void setSource( String pom )
> +    {
> +        _sourceDependency = createDependency();
> +
> +        if( getId() != null  )
> +            _sourceDependency.setId( getId() );
> +
> +        _sourceDependency.setPom( pom );
> +    }
[SNAP]

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


Mime
View raw message