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

Re: Patch - unnecessary actions

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: Sun, 07 Aug 2011 22:25:02 +0200

On 07.08.2011 22:23, Oto BREZINA wrote:
> On 2011-08-04 18:44, Stefan Küng wrote:
>>
>>> Side question how good is VS to use merged strings? From my expirence
>>> using single object (CString) save some conversions and space. This is
>>> minor trought.
>> Can you explain in more detail what you want to do?
> You have many line containing something like:
> String += '\t';
>
> In some languages and compilers there is not direct String + char*
> this make string object creation for '\t' every time you use it. It may
> be marginaly improvement to use String + String. However I use VC only
> for TSVN.

If there was a performance problem here, then of course I'd be ok with
trying to optimize it. But in this case, there's no performance problem.
So I'd like to keep the code as it is - it's easier to read and
understand this way.

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=2812360
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2011-08-07 22:25:28 CEST

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.