Re: [Libreoffice] [Patch] Unreachable code ?

Ivan Timofeev <timofeev.i.s -AT- gmail.com>
Fri, 18 Nov 2011 15:49:11 +0400

Hi Muthu,

18.11.2011 15:38, Muthu Subramanian K пишет:

That code is inside the constructor.
and the constructor sets the variable pUndoSet to NULL right on the top.
So, the if would always fail! Unless I am missing something. Looks too
trivial to be true ;)

I don't see any way to assign non-NULL value to pUndoSet prior to thatif. Most likely we can remove it...


Regards,
    Ivan

Context

Re: [Libreoffice] Extension dependencies (continued)
- Re: [Libreoffice] Extension dependencies · Christophe Strobbe
  - Re: [Libreoffice] Extension dependencies · Stephan Bergmann
    - Re: [Libreoffice] Extension dependencies · Christophe Strobbe
      - Re: [Libreoffice] Extension dependencies · Stephan Bergmann
        
        Re: [Libreoffice] Extension dependencies · Christophe Strobbe
        
        Re: [Libreoffice] Extension dependencies · Stephan Bergmann
- [Libreoffice] [Patch] Unreachable code ? · Muthu Subramanian K
  - Re: [Libreoffice] [Patch] Unreachable code ? · Michael Meeks
    - Re: [Libreoffice] [Patch] Unreachable code ? · Michael Stahl
    - Re: [Libreoffice] [Patch] Unreachable code ? · Muthu Subramanian K
      - Re: [Libreoffice] [Patch] Unreachable code ? · Ivan Timofeev
        
        Re: [Libreoffice] [Patch] Unreachable code ? · Terrence Enger
  - [Libreoffice] [PUSHED][Patch] Unreachable code ? · Caolán McNamara

Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.