Cleaned up no more useful TODOs

git-svn-id: svn+ssh://svn.code.sf.net/p/supertuxkart/code/main/trunk@5106 178a84e3-b1eb-0310-8ba1-8eac791a3b58

src/challenges/challenge_data.cpp

@@ -230,7 +230,7 @@ void ChallengeData::getUnlocks(const XMLNode *root, const std:: string type,
                             }
     case UNLOCK_DIFFICULTY:
                             {
-                            irr::core::stringw user_name = "?"; // TODO
+                            irr::core::stringw user_name = "?"; //TODO: difficulty names when unlocking
                             addUnlockDifficultyReward(attrib, user_name);
                             break;
                             }

src/guiengine/skin.cpp

@@ -638,7 +638,8 @@ void Skin::drawRibbon(const core::rect< s32 > &rect, Widget* widget, const bool
 
 void Skin::drawRibbonChild(const core::rect< s32 > &rect, Widget* widget, const bool pressed, bool focused)
 {
-    //TODO: do something better than hide it when a widget is deactivated?
+    // for now, when this kind of widget is disabled, just hide it. we can change that behaviour if
+    // we ever need to...
     if (widget->m_deactivated) return;
     
     const int playerID = 0; // FIXME : don't hardcode player 0 ?
@@ -957,7 +958,8 @@ void Skin::drawSpinnerChild(const core::rect< s32 > &rect, Widget* widget, const
 
 void Skin::drawIconButton(const core::rect< s32 > &rect, Widget* widget, const bool pressed, bool focused)
 {
-    //TODO: do something better than hide it when a widget is deactivated?
+    // for now, when this kind of widget is disabled, just hide it. we can change that behaviour if
+    // we ever need to...
     if (widget->m_deactivated) return;
     
     
@@ -1024,7 +1026,8 @@ void Skin::drawIconButton(const core::rect< s32 > &rect, Widget* widget, const b
 
 void Skin::drawCheckBox(const core::rect< s32 > &rect, Widget* widget, bool focused)
 { 
-    //TODO: do something better than hide it when a widget is deactivated?
+    // for now, when this kind of widget is disabled, just hide it. we can change that behaviour if
+    // we ever need to...
     if (widget->m_deactivated) return;
     
     CheckBoxWidget* w = dynamic_cast<CheckBoxWidget*>(widget);
@@ -1123,14 +1126,7 @@ void Skin::drawScrollbarThumb(const irr::core::rect< irr::s32 > &rect)
 void Skin::drawScrollbarButton(const irr::core::rect< irr::s32 > &rect, const bool pressed,
                                const bool bottomArrow)
 {
-    /*
+
-    //TODO: allow skinning scrollbar
-    GUIEngine::getDriver()->draw2DRectangle( (pressed ?
-                                              video::SColor(255, 0,   175, 0) :
-                                              video::SColor(255, 150, 150, 150)),
-                                             rect );
-     */
-    
     if (pressed)
     {
         BoxRenderParams& p = SkinConfig::m_render_params["scrollbar_button::down"];

src/guiengine/widgets/label_widget.cpp

@@ -40,7 +40,7 @@ void LabelWidget::add()
     EGUI_ALIGNMENT align = EGUIA_UPPERLEFT;
     if      (m_properties[PROP_TEXT_ALIGN] == "center") align = EGUIA_CENTER;
     else if (m_properties[PROP_TEXT_ALIGN] == "right")  align = EGUIA_LOWERRIGHT;
-    EGUI_ALIGNMENT valign = EGUIA_CENTER ; // TODO - make confiurable through XML file?
+    EGUI_ALIGNMENT valign = EGUIA_CENTER ; //TODO: make label v-align configurable through XML file?
     
     IGUIStaticText* irrwidget = GUIEngine::getGUIEnv()->addStaticText(message.c_str(), widget_size, false, word_wrap, m_parent, -1);
     m_element = irrwidget;

src/input/input_manager.cpp

@@ -164,7 +164,7 @@ void InputManager::handleStaticAction(int key, int value)
             // to reload all textures, display lists etc. Fullscreen can
             // be toggled from the main menu (options->display).
         case KEY_F9:
-            // TODO
+            // TODO; toggle fullscreen
             //irrDriver->toggleFullscreen(false);   // 0: do not reset textures
             // Fall through to put the game into pause mode.
 #endif

src/states_screens/arenas_screen.cpp

@@ -52,7 +52,7 @@ void ArenasScreen::eventCallback(Widget* widget, const std::string& name, const
 
     if (selection == "random_track")
     {
-        // TODO
+        // TODO: random arena selection
     }
     else
     {

src/states_screens/feature_unlocked.cpp

@@ -322,7 +322,7 @@ void FeatureUnlockedCutScene::addUnlockedThings(const std::vector<const Challeng
                 }
                 case UNLOCK_GP:
                 {
-                    //TODO: implement
+                    //TODO: implement gp reward
                     std::cerr << "OK, I see you unlocked a GP, but this is not supported yet\n";
                     
                     video::ITexture* tex = irr_driver->getTexture( file_manager->getGUIDir() + "/main_help.png");
@@ -351,7 +351,7 @@ void FeatureUnlockedCutScene::addUnlockedThings(const std::vector<const Challeng
                 }
                 case UNLOCK_DIFFICULTY:
                 {
-                    //TODO : implement
+                    //TODO : implement difficulty reward
                     std::cerr << "OK, I see you unlocked a difficulty, but this is not supported yet\n";
 
                     video::ITexture* tex = irr_driver->getTexture( file_manager->getGUIDir() + "/main_help.png");

src/states_screens/options_screen_players.cpp

@@ -98,9 +98,6 @@ bool OptionsScreenPlayers::gotNewPlayerName(const stringw& newName, PlayerProfil
         }
         
     }
-    
-    // TODO : need to re-save user config here?
-    
     return true;
 }
 
@@ -120,8 +117,6 @@ void OptionsScreenPlayers::deletePlayer(PlayerProfile* player)
     {
         players->addItem(UserConfigParams::m_all_players[n].getName());
     }
-    
-    // TODO : need to re-save user config here?
 }
 
 

src/states_screens/tracks_screen.cpp

@@ -181,7 +181,6 @@ void TracksScreen::init()
             Track* curr = track_manager->getTrack(tracks[t]);
             if (curr == NULL)
             {
-                //TODO: show warning to user? don't add GP to GUI?
                 std::cerr << "/!\\ WARNING: Grand Prix '" << gp->getId() << "' refers to track '"
                           << tracks[t] << "', which does not exist.\n";
             }
@@ -192,7 +191,6 @@ void TracksScreen::init()
         }
         if (sshot_files.size() == 0)
         {
-            //TODO: show warning to user? don't add GP to GUI?
             std::cerr << "/!\\ WARNING: Grand Prix '" << gp->getId() << "' does not contain any valid track.\n";
             sshot_files.push_back("gui/main_help.png");
         }