Review Request: delete some unneded if(not null)

[David Jarvie]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/5021/#review7209
-----------------------------------------------------------


Looks good to me.

- David


On 2010-08-25 09:21:53, Jaime Torres wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/5021/
> -----------------------------------------------------------
> 
> (Updated 2010-08-25 09:21:53)
> 
> 
> Review request for kdelibs.
> 
> 
> Summary
> -------
> 
> delete handles null pointers fine in c++
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdelibs/dnssd/mdnsd-responder.cpp 1167520 
>   /trunk/KDE/kdelibs/kate/syntax/katesyntaxdocument.cpp 1167520 
>   /trunk/KDE/kdelibs/kde3support/kdeui/k3dockwidget_private.cpp 1167520 
>   /trunk/KDE/kdelibs/kdecore/date/ktimezone.cpp 1167520 
>   /trunk/KDE/kdelibs/kdecore/kernel/kcmdlineargs.cpp 1167520 
>   /trunk/KDE/kdelibs/kdecore/localization/guess_ja_p.h 1167520 
>   /trunk/KDE/kdelibs/kdecore/localization/probers/nsUniversalDetector.cpp 1167520 
>   /trunk/KDE/kdelibs/kdeui/kernel/kapplication.cpp 1167520 
>   /trunk/KDE/kdelibs/kdeui/tests/ktabwidgettest.cpp 1167520 
>   /trunk/KDE/kdelibs/kdeui/widgets/kled.cpp 1167520 
>   /trunk/KDE/kdelibs/kio/kssl/ksslpkcs12.cpp 1167520 
>   /trunk/KDE/kdelibs/kio/kssl/ksslpkcs7.cpp 1167520 
>   /trunk/KDE/kdelibs/kio/misc/ksendbugmail/smtp.cpp 1167520 
>   /trunk/KDE/kdelibs/kioslave/http/http.cpp 1167520 
>   /trunk/KDE/kdelibs/plasma/abstractrunner.cpp 1167520 
>   /trunk/KDE/kdelibs/plasma/containment.cpp 1167520 
>   /trunk/KDE/kdelibs/plasma/packagestructure.cpp 1167520 
>   /trunk/KDE/kdelibs/plasma/service.cpp 1167520 
> 
> Diff: http://reviewboard.kde.org/r/5021/diff
> 
> 
> Testing
> -------
> 
> working with it.
> 
> 
> Thanks,
> 
> Jaime
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20100825/29582598/attachment.htm>