Login | Register
My pages Projects Community openCollabNet

Discussions > dev > Re: Patch - unnecessary actions

Project highlights: :. Download .: :. Support .: :. FAQ .: :. Translations .: :. Donate .: :. Report Bug .:

tortoisesvn
Discussion topic

Back to topic list

Re: Patch - unnecessary actions

Author steveking
Full name Stefan Küng
Date 2011-08-07 13:25:27 PDT
Message 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

« Previous message in topic | 5 of 6 | Next message in topic »

Messages

Show all messages in topic

Patch - unnecessary actions wipedout Dmitry 2011-08-02 23:05:35 PDT
     Re: Patch - unnecessary actions otik Oto BREZINA 2011-08-03 15:26:48 PDT
         Re: Patch - unnecessary actions steveking Stefan Küng 2011-08-04 09:45:27 PDT
             Re: Patch - unnecessary actions otik Oto BREZINA 2011-08-07 13:23:19 PDT
                 Re: Patch - unnecessary actions steveking Stefan Küng 2011-08-07 13:25:27 PDT
                 RE: Re: Patch - unnecessary actions klimax Daniel Klima 2011-08-08 00:25:58 PDT
Messages per page: