[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Comment on revision r25052 in tortoisesvn

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: Sat, 28 Dec 2013 09:01:18 +0100

On 28.12.2013 08:54, Oto BREZINA wrote:
> On 2013-12-26 18:26, tortoisesvn_at_googlecode.com wrote:
>> Comment by XhmikosR:
>>
>>
>> In line `return x % m_nTabSize ? m_nTabSize - (x % m_nTabSize) :
>> m_nTabSize;`, can you add parentheses to clarify the expression?
> Does it make sense at all?
> m_nTabSize - (x % m_nTabSize)
> seems to be enought.
>

I thought so too, but apparently it's not clear enough for everyone.
So it doesn't hurt to add parenthesis to ensure everyone gets it.

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest interface to (Sub)version control
    /_/   \_\     http://tortoisesvn.net
------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=3070878
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2013-12-28 09:01:36 CET

This is an archived mail posted to the TortoiseSVN Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.