adjust indentation of code from previous patches and make shift-tab customizable

doc/keys

@@ -56,6 +56,8 @@
 #
 #key_screen_switcher = 9
 #
+#key_backward_screen_switcher = 353
+#
 #key_help = '1' 265
 #
 #key_playlist = '2' 266

src/help.cpp

@@ -101,6 +101,8 @@ std::string Help::DisplayKeys(int *key, int size)
 			result += "Left";
 		else if (key[i] == 9)
 			result += "Tab";
+		else if (key[i] == 353)
+			result += "Shift-Tab";
 		else if (key[i] >= 1 && key[i] <= 26)
 		{
 			result += "Ctrl-";
@@ -145,7 +147,10 @@ void Help::GetKeybindings()
 	if (Config.screen_switcher_previous)
 		*w << DisplayKeys(Key.ScreenSwitcher)   << "Switch between current and last screen\n";
 	else
-		*w << DisplayKeys(Key.ScreenSwitcher)   << "Switch between given sequence of screens\n";
+	{
+		*w << DisplayKeys(Key.ScreenSwitcher)   << "Switch to next screen in sequence\n";
+		*w << DisplayKeys(Key.BackwardScreenSwitcher) << "Switch to previous screen in sequence\n";
+	}
 	*w << DisplayKeys(Key.Help)			<< "Help screen\n";
 	*w << DisplayKeys(Key.Playlist)			<< "Playlist screen\n";
 	*w << DisplayKeys(Key.Browser)			<< "Browse screen\n";

src/ncmpcpp.cpp

@@ -2231,7 +2231,7 @@ int main(int argc, char *argv[])
 					(*screen)->SwitchTo();
 			}
 		}
-                else if (input == 353)
+		else if (!Config.screen_switcher_previous && Keypressed(input, Key.BackwardScreenSwitcher))
 		{
 			if (Config.screen_switcher_previous)
 			{
@@ -2243,14 +2243,10 @@ int main(int argc, char *argv[])
 			else if (!Config.screens_seq.empty())
 			{
 				std::list<BasicScreen *>::const_iterator screen = std::find(Config.screens_seq.begin(), Config.screens_seq.end(), myScreen);
-				if (screen == Config.screens_seq.begin()) {
-                                        screen = --(Config.screens_seq.end());
-                                        (*screen)->SwitchTo();
-                                }
-				else {
-                                        --screen;
-					(*screen)->SwitchTo();
-                                }
+				if (screen == Config.screens_seq.begin())
+					(*Config.screens_seq.rbegin())->SwitchTo();
+				else
+					(*--screen)->SwitchTo();
 			}
 		}
 		else if (Keypressed(input, Key.Playlist))

src/settings.cpp

@@ -185,6 +185,7 @@ void NcmpcppKeys::SetDefaults()
 	PrevColumn[0] = KEY_LEFT;
 	NextColumn[0] = KEY_RIGHT;
 	ScreenSwitcher[0] = 9;
+	BackwardScreenSwitcher[0] = 353;
 	Help[0] = '1';
 	Playlist[0] = '2';
 	Browser[0] = '3';
@@ -272,6 +273,7 @@ void NcmpcppKeys::SetDefaults()
 	PrevColumn[1] = NullKey;
 	NextColumn[1] = NullKey;
 	ScreenSwitcher[1] = NullKey;
+	BackwardScreenSwitcher[1] = NullKey;
 	Help[1] = 265;
 	Playlist[1] = 266;
 	Browser[1] = 267;
@@ -502,6 +504,8 @@ void NcmpcppKeys::Read()
 				GetKeys(key, NextColumn);
 			else if (key.find("key_screen_switcher ") != std::string::npos)
 				GetKeys(key, ScreenSwitcher);
+			else if (key.find("key_backward_screen_switcher ") != std::string::npos)
+				GetKeys(key, BackwardScreenSwitcher);
 			else if (key.find("key_help ") != std::string::npos)
 				GetKeys(key, Help);
 			else if (key.find("key_playlist ") != std::string::npos)

src/settings.h

@@ -76,6 +76,7 @@ struct NcmpcppKeys
 	int PrevColumn[2];
 	int NextColumn[2];
 	int ScreenSwitcher[2];
+	int BackwardScreenSwitcher[2];
 	int Help[2];
 	int Playlist[2];
 	int Browser[2];