extragear/multimedia/amarok/src

Jeff Mitchell kde-dev at emailgoeshere.com
Wed Sep 20 02:04:58 UTC 2006


I just caught this, when reviewing all the SVN commits from when I was gone:

AFAIK, we should actually not be comparing to QString::isNull(); we  
should be comparing to QString::isEmpty().  The former case does not  
handle a non-null but empty string ( "" ), while the latter case  
handles both.  Unless there's a reason we might want to allow an empty  
string in the comparison.

--Jeff

Quoting Alexandre Pereira de Oliveira <aoliveira at kdemail.net>:

> SVN commit 583697 by aoliveira:
>
> Still on englishbreakfastnetwork suggested fixes
>  Still some " " to ' ' changes;
>  Use QString::isNull() instead of comparing to QString::null;
>  Some typos on comments;
>  Some const correctness;
>
>
>  M  +1 -1      amarokcore/crashhandler.cpp
>  M  +2 -2      app.cpp
>  M  +1 -1      atomicurl.cpp
>  M  +5 -5      collectiondb.cpp
>  M  +1 -1      collectiondb.h
>  M  +1 -1      database_refactor/collectiondb.cpp
>  M  +5 -5      devicemanager.cpp
>  M  +5 -5      devicemanager.h
>  M  +1 -1      engine/gst10/gstengine.cpp
>  M  +128 -128  engine/helix/helix-sp/helix-sp.cpp
>  M  +14 -14    engine/helix/helix-sp/hspcontext.cpp
>  M  +7 -7      engine/helix/helix-sp/hsphook.cpp
>  M  +7 -7      engine/nmm/HostListItem.cpp
>  M  +2 -2      ktrm.cpp
>  M  +3 -3      lastfm.cpp
>  M  +3 -3      lastfm.h
>  M  +4 -4      mediabrowser.cpp
>  M  +1 -1      mediadevice/daap/daapclient.cpp
>  M  +2 -2      mediadevice/generic/genericmediadevice.cpp
>  M  +3 -3      mediadevice/ipod/ipodmediadevice.cpp
>  M  +4 -4      mediumpluginmanager.cpp
>  M  +1 -1      metabundle.cpp
>  M  +3 -3      metadata/m4a/itunesalbbox.cpp
>  M  +3 -3      metadata/m4a/itunesartbox.cpp
>  M  +3 -3      metadata/m4a/itunescmtbox.cpp
>  M  +3 -3      metadata/m4a/itunescvrbox.cpp
>  M  +3 -3      metadata/m4a/itunesdaybox.cpp
>  M  +3 -3      metadata/m4a/itunesdiskbox.cpp
>  M  +3 -3      metadata/m4a/itunesgenbox.cpp
>  M  +3 -3      metadata/m4a/itunesgrpbox.cpp
>  M  +3 -3      metadata/m4a/itunesnambox.cpp
>  M  +3 -3      metadata/m4a/itunestmpobox.cpp
>  M  +3 -3      metadata/m4a/itunestrknbox.cpp
>  M  +3 -3      metadata/m4a/ituneswrtbox.cpp
>  M  +1 -1      osd.cpp
>  M  +1 -1      playlist.cpp
>  M  +1 -1      playlist.h
>  M  +3 -3      playlistbrowser.cpp
>  M  +3 -3      playlistbrowseritem.cpp
>  M  +4 -4      scancontroller.cpp
>  M  +1 -1      smartplaylisteditor.cpp
>  M  +1 -1      statusbar/statusbar.cpp
>  M  +2 -2      vis/xmmswrapper/xmmswrapper.cpp
>
>
>





More information about the Amarok mailing list