author | erik <erik> | 2007-01-31 22:23:27 (UTC) |
---|---|---|
committer | erik <erik> | 2007-01-31 22:23:27 (UTC) |
commit | 73edac572d9ac9cfd18cc1254fc2019563d55f16 (patch) (side-by-side diff) | |
tree | a420b341ce2bcb4a049c76935544b8bce2a0a853 /noncore/multimedia/opieplayer2/playlistselection.cpp | |
parent | ed5fba8cdb27174427b2e7780508bafa05536ca1 (diff) | |
download | opie-73edac572d9ac9cfd18cc1254fc2019563d55f16.zip opie-73edac572d9ac9cfd18cc1254fc2019563d55f16.tar.gz opie-73edac572d9ac9cfd18cc1254fc2019563d55f16.tar.bz2 |
A few files that dereference pointers w/out checking that the pointer is
valid.
Diffstat (limited to 'noncore/multimedia/opieplayer2/playlistselection.cpp') (more/less context) (ignore whitespace changes)
-rw-r--r-- | noncore/multimedia/opieplayer2/playlistselection.cpp | 35 |
1 files changed, 10 insertions, 25 deletions
diff --git a/noncore/multimedia/opieplayer2/playlistselection.cpp b/noncore/multimedia/opieplayer2/playlistselection.cpp index 678ebdf..780d705 100644 --- a/noncore/multimedia/opieplayer2/playlistselection.cpp +++ b/noncore/multimedia/opieplayer2/playlistselection.cpp @@ -52,10 +52,2 @@ PlayListSelection::PlayListSelection( QWidget *parent, const char *name ) // odebug << "starting playlistselector" << oendl; -// #ifdef USE_PLAYLIST_BACKGROUND -// setStaticBackground( TRUE ); -// setBackgroundPixmap( Resource::loadPixmap( "opieplayer/background" ) ); - -// setBackgroundPixmap( Resource::loadPixmap( "launcher/opielogo" ) ); -// #endif -// addColumn("Title",236); -// setAllColumnsShowFocus( TRUE ); addColumn( tr( "Playlist Selection" ) ); @@ -70,3 +62,2 @@ PlayListSelection::~PlayListSelection() { -// #ifdef USE_PLAYLIST_BACKGROUND void PlayListSelection::drawBackground( QPainter *p, const QRect &r ) { @@ -74,7 +65,3 @@ void PlayListSelection::drawBackground( QPainter *p, const QRect &r ) { p->fillRect( r, QBrush( white ) ); -// QImage logo = Resource::loadImage( "launcher/opielogo" ); -// if ( !logo.isNull() ) -// p->drawImage( (width() - logo.width()) / 2, (height() - logo.height()) / 2, logo ); } -// #endif @@ -96,3 +83,3 @@ const DocLnk *PlayListSelection::current() { if ( item ) - return item->file(); + return item->file(); return NULL; @@ -122,3 +109,3 @@ void PlayListSelection::moveSelectedUp() { if ( item && item->itemAbove() ) - item->itemAbove()->moveItem( item ); + item->itemAbove()->moveItem( item ); ensureItemVisible( selectedItem() ); @@ -140,3 +127,3 @@ bool PlayListSelection::prev() { else - return FALSE; + return FALSE; ensureItemVisible( selectedItem() ); @@ -150,3 +137,3 @@ bool PlayListSelection::next() { else - return FALSE; + return FALSE; ensureItemVisible( selectedItem() ); @@ -161,3 +148,3 @@ bool PlayListSelection::first() { else - return FALSE; + return FALSE; ensureItemVisible( selectedItem() ); @@ -171,3 +158,3 @@ bool PlayListSelection::last() { while ( ( item = item->nextSibling() ) ) - prevItem = item; + prevItem = item; if ( prevItem ) @@ -175,3 +162,3 @@ bool PlayListSelection::last() { else - return FALSE; + return FALSE; ensureItemVisible( selectedItem() ); @@ -182,3 +169,2 @@ void PlayListSelection::unSelect() { - //QListViewItem *item = selectedItem(); setSelected( currentItem(), FALSE); @@ -189,5 +175,6 @@ void PlayListSelection::writeCurrent( Config& cfg ) { QListViewItem *item = selectedItem(); - if ( item ) + if ( item ) { cfg.writeEntry("current", item->text(0) ); - odebug << item->text(0) << oendl; + odebug << item->text(0) << oendl; + } @@ -208,4 +195,2 @@ void PlayListSelection::setSelectedItem(const QString &strk ) { } -// setSelected( item, TRUE ); -// ensureItemVisible( selectedItem() ); } |