Isn’t this checkstyle’s default? In general when I coded the first 
checkstyle.xml I tried to follow the default as much as possible since the 
checkstyle’s team has usually defined sensible defaults after comparing various 
projects.

Are you sure we’re checking this in checkstyle btw? I’m asking because I 
remember using it without spaces recently and checkstyle not complaining

Otherwise I’m +0 to change it (where is it defined?).

Thanks
-Vincent


On 18 Jul 2014 at 18:38:41, Sergiu Dumitriu 
([email protected](mailto:[email protected])) wrote:

> Hi devs,
>  
> The current codestyle for wildcard type parameters is to use spaces
> around it, as in:
>  
> List< ? >
> Map< ? , ? >
> Map
>  
> As opposed to specific parameters:
>  
> List
> List
> Map
> Map
>  
> This might have a historical reason, when we defined the codestyle,
> Eclipse couldn't make the difference between "?" as a wildcard type and
> "?" as the ternary operator.
>  
> The standard Java, Eclipse, and IDEA style conventions don't use the
> extra space.
>  
> Personally, I don't think that the wildcard needs the extra space, so +1
> for this change.
>  
> --
> Sergiu Dumitriu
> http://purl.org/net/sergiu
_______________________________________________
devs mailing list
[email protected]
http://lists.xwiki.org/mailman/listinfo/devs

Reply via email to