Re: Patch: Remove redundant checks

On 23.12.2009 09:33, Dmitry wrote:
> Stefan wrote:
>>> Also in TortoiseProc/Revisio​nGraph/SearchPathTre​e.cpp I found
>>> some strange code pattern that I marked with a comment. I suggest
>>> it is reviewed - looks like a severe bug.
>> That code is actually correct.
> Then I don't get what is meant there. What I see is the following: if
> "firstChild" is a null pointer nothing happens, but if it is not a
> null pointer "delete firstChild" is done, "firstChild" is not changed
> and remains not-null, so the while-loop continues forever and each
> iteration starting with the second one performs "delete" on an
> already deleted pointer. Did I miss anything?

The 'delete firstChild' triggers the destructor of 'firstChild' which
takes care of the rest. Just step through that part of the code with the
debugger and you'll see.

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=2432665
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].