author | simon <simon> | 2002-12-02 17:10:14 (UTC) |
---|---|---|
committer | simon <simon> | 2002-12-02 17:10:14 (UTC) |
commit | e2e40368d1e98cfd3814e14a56c0e503d494cf4f (patch) (unidiff) | |
tree | 2dc59b9f4e5c5a8cd971934b1dd5976939bbac37 | |
parent | d86803bacac02ae99388cba069532851a5e08ce8 (diff) | |
download | opie-e2e40368d1e98cfd3814e14a56c0e503d494cf4f.zip opie-e2e40368d1e98cfd3814e14a56c0e503d494cf4f.tar.gz opie-e2e40368d1e98cfd3814e14a56c0e503d494cf4f.tar.bz2 |
- removed unused (and never initialized to something != 0)
PlayListWidgetPrivate::current
-rw-r--r-- | noncore/multimedia/opieplayer2/playlistwidget.cpp | 7 | ||||
-rw-r--r-- | noncore/multimedia/opieplayer2/playlistwidgetgui.cpp | 1 | ||||
-rw-r--r-- | noncore/multimedia/opieplayer2/playlistwidgetgui.h | 1 |
3 files changed, 1 insertions, 8 deletions
diff --git a/noncore/multimedia/opieplayer2/playlistwidget.cpp b/noncore/multimedia/opieplayer2/playlistwidget.cpp index 55b66ab..6355bec 100644 --- a/noncore/multimedia/opieplayer2/playlistwidget.cpp +++ b/noncore/multimedia/opieplayer2/playlistwidget.cpp | |||
@@ -160,27 +160,24 @@ PlayListWidget::PlayListWidget( QWidget* parent, const char* name, WFlags fl ) | |||
160 | audioPopulated=false; | 160 | audioPopulated=false; |
161 | videoPopulated=false; | 161 | videoPopulated=false; |
162 | populateSkinsMenu(); | 162 | populateSkinsMenu(); |
163 | initializeStates(); | 163 | initializeStates(); |
164 | 164 | ||
165 | cfg.setGroup("PlayList"); | 165 | cfg.setGroup("PlayList"); |
166 | QString currentPlaylist = cfg.readEntry( "CurrentPlaylist", "default"); | 166 | QString currentPlaylist = cfg.readEntry( "CurrentPlaylist", "default"); |
167 | loadList(DocLnk( currentPlaylist ) ); | 167 | loadList(DocLnk( currentPlaylist ) ); |
168 | } | 168 | } |
169 | 169 | ||
170 | 170 | ||
171 | PlayListWidget::~PlayListWidget() { | 171 | PlayListWidget::~PlayListWidget() { |
172 | if ( d->current ) { | ||
173 | delete d->current; | ||
174 | } | ||
175 | delete d; | 172 | delete d; |
176 | } | 173 | } |
177 | 174 | ||
178 | 175 | ||
179 | void PlayListWidget::initializeStates() { | 176 | void PlayListWidget::initializeStates() { |
180 | d->tbPlay->setOn( mediaPlayerState->isPlaying() ); | 177 | d->tbPlay->setOn( mediaPlayerState->isPlaying() ); |
181 | d->tbLoop->setOn( mediaPlayerState->isLooping() ); | 178 | d->tbLoop->setOn( mediaPlayerState->isLooping() ); |
182 | d->tbShuffle->setOn( mediaPlayerState->isShuffled() ); | 179 | d->tbShuffle->setOn( mediaPlayerState->isShuffled() ); |
183 | setPlaylist( true ); | 180 | setPlaylist( true ); |
184 | } | 181 | } |
185 | 182 | ||
186 | void PlayListWidget::writeDefaultPlaylist() { | 183 | void PlayListWidget::writeDefaultPlaylist() { |
@@ -419,28 +416,26 @@ void PlayListWidget::setDocument( const QString& fileref ) { | |||
419 | } | 416 | } |
420 | 417 | ||
421 | 418 | ||
422 | void PlayListWidget::useSelectedDocument() { | 419 | void PlayListWidget::useSelectedDocument() { |
423 | d->setDocumentUsed = FALSE; | 420 | d->setDocumentUsed = FALSE; |
424 | } | 421 | } |
425 | 422 | ||
426 | 423 | ||
427 | const DocLnk *PlayListWidget::current() const { // this is fugly | 424 | const DocLnk *PlayListWidget::current() const { // this is fugly |
428 | assert( currentTab() == CurrentPlayList ); | 425 | assert( currentTab() == CurrentPlayList ); |
429 | 426 | ||
430 | // qDebug("playlist"); | 427 | // qDebug("playlist"); |
431 | if ( mediaPlayerState->isUsingPlaylist() ) { | 428 | if ( mediaPlayerState->isUsingPlaylist() ) |
432 | return d->selectedFiles->current(); | 429 | return d->selectedFiles->current(); |
433 | } else if ( d->setDocumentUsed && d->current ) | ||
434 | return d->current; | ||
435 | 430 | ||
436 | assert( false ); | 431 | assert( false ); |
437 | } | 432 | } |
438 | 433 | ||
439 | 434 | ||
440 | bool PlayListWidget::prev() { | 435 | bool PlayListWidget::prev() { |
441 | if ( mediaPlayerState->isUsingPlaylist() ) { | 436 | if ( mediaPlayerState->isUsingPlaylist() ) { |
442 | if ( mediaPlayerState->isShuffled() ) { | 437 | if ( mediaPlayerState->isShuffled() ) { |
443 | const DocLnk *cur = current(); | 438 | const DocLnk *cur = current(); |
444 | int j = 1 + (int)(97.0 * rand() / (RAND_MAX + 1.0)); | 439 | int j = 1 + (int)(97.0 * rand() / (RAND_MAX + 1.0)); |
445 | for ( int i = 0; i < j; i++ ) { | 440 | for ( int i = 0; i < j; i++ ) { |
446 | if ( !d->selectedFiles->next() ) | 441 | if ( !d->selectedFiles->next() ) |
diff --git a/noncore/multimedia/opieplayer2/playlistwidgetgui.cpp b/noncore/multimedia/opieplayer2/playlistwidgetgui.cpp index a217c94..db7c979 100644 --- a/noncore/multimedia/opieplayer2/playlistwidgetgui.cpp +++ b/noncore/multimedia/opieplayer2/playlistwidgetgui.cpp | |||
@@ -57,25 +57,24 @@ | |||
57 | 57 | ||
58 | #include "audiowidget.h" | 58 | #include "audiowidget.h" |
59 | #include "videowidget.h" | 59 | #include "videowidget.h" |
60 | #include "mediaplayerstate.h" | 60 | #include "mediaplayerstate.h" |
61 | 61 | ||
62 | extern MediaPlayerState *mediaPlayerState; | 62 | extern MediaPlayerState *mediaPlayerState; |
63 | 63 | ||
64 | PlayListWidgetGui::PlayListWidgetGui( QWidget* parent, const char* name, WFlags fl ) | 64 | PlayListWidgetGui::PlayListWidgetGui( QWidget* parent, const char* name, WFlags fl ) |
65 | : QMainWindow( parent, name, fl ) { | 65 | : QMainWindow( parent, name, fl ) { |
66 | 66 | ||
67 | d = new PlayListWidgetPrivate; | 67 | d = new PlayListWidgetPrivate; |
68 | d->setDocumentUsed = FALSE; | 68 | d->setDocumentUsed = FALSE; |
69 | d->current = NULL; | ||
70 | 69 | ||
71 | setBackgroundMode( PaletteButton ); | 70 | setBackgroundMode( PaletteButton ); |
72 | setToolBarsMovable( FALSE ); | 71 | setToolBarsMovable( FALSE ); |
73 | 72 | ||
74 | // Create Toolbar | 73 | // Create Toolbar |
75 | QPEToolBar *toolbar = new QPEToolBar( this ); | 74 | QPEToolBar *toolbar = new QPEToolBar( this ); |
76 | toolbar->setHorizontalStretchable( TRUE ); | 75 | toolbar->setHorizontalStretchable( TRUE ); |
77 | 76 | ||
78 | // Create Menubar | 77 | // Create Menubar |
79 | QPEMenuBar *menu = new QPEMenuBar( toolbar ); | 78 | QPEMenuBar *menu = new QPEMenuBar( toolbar ); |
80 | menu->setMargin( 0 ); | 79 | menu->setMargin( 0 ); |
81 | 80 | ||
diff --git a/noncore/multimedia/opieplayer2/playlistwidgetgui.h b/noncore/multimedia/opieplayer2/playlistwidgetgui.h index 6c35771..4eefb93 100644 --- a/noncore/multimedia/opieplayer2/playlistwidgetgui.h +++ b/noncore/multimedia/opieplayer2/playlistwidgetgui.h | |||
@@ -55,25 +55,24 @@ class QPEToolBar; | |||
55 | class QListViewItem; | 55 | class QListViewItem; |
56 | class QListView; | 56 | class QListView; |
57 | class QPoint; | 57 | class QPoint; |
58 | class QAction; | 58 | class QAction; |
59 | class QLabel; | 59 | class QLabel; |
60 | 60 | ||
61 | class PlayListWidgetPrivate { | 61 | class PlayListWidgetPrivate { |
62 | public: | 62 | public: |
63 | QToolButton *tbPlay, *tbFull, *tbLoop, *tbShuffle, *tbAddToList, *tbRemoveFromList, *tbMoveUp, *tbMoveDown, *tbRemove; | 63 | QToolButton *tbPlay, *tbFull, *tbLoop, *tbShuffle, *tbAddToList, *tbRemoveFromList, *tbMoveUp, *tbMoveDown, *tbRemove; |
64 | QFrame *playListFrame; | 64 | QFrame *playListFrame; |
65 | PlayListSelection *selectedFiles; | 65 | PlayListSelection *selectedFiles; |
66 | bool setDocumentUsed; | 66 | bool setDocumentUsed; |
67 | DocLnk *current; | ||
68 | }; | 67 | }; |
69 | 68 | ||
70 | 69 | ||
71 | class ToolButton : public QToolButton { | 70 | class ToolButton : public QToolButton { |
72 | public: | 71 | public: |
73 | ToolButton( QWidget *parent, const char *name, const QString& icon, QObject *handler, const QString& slot, bool t = FALSE ) | 72 | ToolButton( QWidget *parent, const char *name, const QString& icon, QObject *handler, const QString& slot, bool t = FALSE ) |
74 | : QToolButton( parent, name ) { | 73 | : QToolButton( parent, name ) { |
75 | setTextLabel( name ); | 74 | setTextLabel( name ); |
76 | setPixmap( Resource::loadPixmap( icon ) ); | 75 | setPixmap( Resource::loadPixmap( icon ) ); |
77 | setAutoRaise( TRUE ); | 76 | setAutoRaise( TRUE ); |
78 | setFocusPolicy( QWidget::NoFocus ); | 77 | setFocusPolicy( QWidget::NoFocus ); |
79 | setToggleButton( t ); | 78 | setToggleButton( t ); |