On Fri, Dec 20, 2013 at 12:21 PM, Emmanuel Bourg <ebo...@apache.org> wrote:

> Le 20/12/2013 18:18, ggreg...@apache.org a écrit :
>
> > Remove some unnecessary parentheses.
>
> I'd argue they make the code easier to read. Reading bit shifting code
> is quite painful, some parentheses help greatly.
>

That's why it's only "some" and not "all" unnecessary parentheses.

IMO:

   tableSize = (1 << 8);

is just clutter, there is no confusion _possible_ about operator precedence
in this line of code.

Gary


> Emmanuel
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> For additional commands, e-mail: dev-h...@commons.apache.org
>
>


-- 
E-Mail: garydgreg...@gmail.com | ggreg...@apache.org
Java Persistence with Hibernate, Second Edition<http://www.manning.com/bauer3/>
JUnit in Action, Second Edition <http://www.manning.com/tahchiev/>
Spring Batch in Action <http://www.manning.com/templier/>
Blog: http://garygregory.wordpress.com
Home: http://garygregory.com/
Tweet! http://twitter.com/GaryGregory

Reply via email to