[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.


   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.net
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.