[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    kdeaddons/noatun-plugins/lyrics
From:       Rob Kaper <cap () capsi ! com>
Date:       2004-01-20 22:00:55
Message-ID: 20040120220055.091291A0E () office ! kde ! org
[Download RAW message or body]

CVS commit by kaper: 

add some safeguards, crashes never make me happy


  M +4 -0      lyrics.cpp   1.29


--- kdeaddons/noatun-plugins/lyrics/lyrics.cpp  #1.28:1.29
@@ -96,4 +96,6 @@ void Lyrics::setProviders( QValueVector<
 
 void Lyrics::attach(bool a) {
+  if ( !napp->player()->current() )
+    return;
   if (a) {
     KMessageBox::information(this, i18n("Choosing this option, the current URL will \
be attached to the current file. This way, if you try to view the lyrics of this file \
later, you won't have to search for it again. This information can be stored between \
sessions, as long as your playlist stores metadata about the multimedia items (almost \
all the playlists do). If you want to be able to search for other lyrics for this \
music, you must select this option again to clear the stored URL."), QString::null, \
"lyrics::attach_info"); @@ -118,4 +120,6 @@ void Lyrics::loadingURL(KIO::Job *)
 void Lyrics::loadedURL()
 {
+  if ( !napp->player()->current() )
+    return;
   statusBar()->changeItem(i18n("Loaded"), 0);
   setCaption(i18n("Lyrics: %1").arg(napp->player()->current().property("title")));


[prev in list] [next in list] [prev in thread] [next in thread]