From 8deaa0a9e3852df2c9c773829555a020578aa62e Mon Sep 17 00:00:00 2001
From: unknown <unknown@localhost.(none)>
Date: Fri, 15 Aug 2008 12:15:34 +0200
Subject: [PATCH] correct playlist clearing

---
 src/menu.cpp           | 3 ++-
 src/menu.h             | 3 +--
 src/ncmpcpp.cpp        | 1 -
 src/status_checker.cpp | 2 +-
 4 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/src/menu.cpp b/src/menu.cpp
index 3df8bb8f..b4892ca4 100644
--- a/src/menu.cpp
+++ b/src/menu.cpp
@@ -529,7 +529,7 @@ void Menu::Highlight(int which)
 	
 	int howmuch = itsBeginning-old_beginning;
 	
-	if (Abs(howmuch) > itsHeight/2)
+	if (Abs(howmuch) > itsHeight)
 		redraw_screen();
 	else
 	{
@@ -561,6 +561,7 @@ void Menu::Clear(bool clear_screen)
 	for (vector<Option *>::iterator it = itsOptions.begin(); it != itsOptions.end(); it++)
 		delete *it;
 	itsOptions.clear();
+	NeedsRedraw.clear();
 	itsStaticsNumber = 0;
 	if (clear_screen)
 		Window::Clear();
diff --git a/src/menu.h b/src/menu.h
index 1f546f39..6f0ba81c 100644
--- a/src/menu.h
+++ b/src/menu.h
@@ -43,7 +43,7 @@ struct Option
 class Menu : public Window
 {
 	public:
-		Menu(int startx, int starty, int width, int height, string title, COLOR color, BORDER border) : Window(startx, starty, width, height, title, color, border), itsStaticsNumber(0), itsChoice(0), itsBeginning(0), itsHighlight(0), itsHighlightColor(itsBaseColor), itsHighlightEnabled(1) { SetColor(color); }
+		Menu(int startx, int starty, int width, int height, string title, COLOR color, BORDER border) : Window(startx, starty, width, height, title, color, border), itsStaticsNumber(0), itsBeginning(0), itsHighlight(0), itsHighlightColor(itsBaseColor), itsHighlightEnabled(1) { SetColor(color); }
 		virtual ~Menu();
 		
 		virtual void Add(string str) { AddOption(str); }
@@ -89,7 +89,6 @@ class Menu : public Window
 		void redraw_screen();
 		bool is_static() { return itsOptions[itsHighlight]->is_static; }
 		
-		int itsChoice;
 		int itsBeginning;
 		int itsHighlight;
 		
diff --git a/src/ncmpcpp.cpp b/src/ncmpcpp.cpp
index 6e95d061..a9f30b5f 100644
--- a/src/ncmpcpp.cpp
+++ b/src/ncmpcpp.cpp
@@ -1525,7 +1525,6 @@ int main(int argc, char *argv[])
 				ShowMessage("Deleting all songs except now playing one...");
 				mpd_playlist_queue_commit(conn);
 				ShowMessage("Songs deleted!");
-				redraw_me = 1;
 				break;
 			}
 			case 'c': // clear playlist
diff --git a/src/status_checker.cpp b/src/status_checker.cpp
index daba7e59..d7a56b2f 100644
--- a/src/status_checker.cpp
+++ b/src/status_checker.cpp
@@ -153,7 +153,7 @@ void NcmpcppStatusChanged(MpdObj *conn, ChangedStatusType what)
 			{
 				if (playlist_length < vPlaylist.size())
 				{
-					mPlaylist->Clear(!playlist_length && current_screen == csPlaylist);
+					mPlaylist->Clear(playlist_length < mPlaylist->GetHeight() && current_screen == csPlaylist);
 					for (vector<Song *>::iterator it = vPlaylist.begin(); it != vPlaylist.end(); it++)
 						delete *it;
 					vPlaylist.clear();