From a3dc17a7dd59b9475ef2aa1d6db4e0bbe885b4a7 Mon Sep 17 00:00:00 2001
From: hikerstk <hikerstk@178a84e3-b1eb-0310-8ba1-8eac791a3b58>
Date: Wed, 25 Aug 2010 13:54:28 +0000
Subject: [PATCH] Fixed handling of ESC (which previously would close the
 screen, without removing the actual race result gui). ESC now allows some
 skipping of animations.

git-svn-id: svn+ssh://svn.code.sf.net/p/supertuxkart/code/main/trunk@5776 178a84e3-b1eb-0310-8ba1-8eac791a3b58
---
 src/states_screens/race_result_gui.cpp | 52 +++++++++++++-------------
 src/states_screens/race_result_gui.hpp | 13 ++-----
 2 files changed, 31 insertions(+), 34 deletions(-)

diff --git a/src/states_screens/race_result_gui.cpp b/src/states_screens/race_result_gui.cpp
index a93c604f9..635c28b52 100644
--- a/src/states_screens/race_result_gui.cpp
+++ b/src/states_screens/race_result_gui.cpp
@@ -42,17 +42,6 @@ RaceResultGUI::RaceResultGUI() : Screen("race_result.stkgui", /*pause race*/ fal
 {
 }   // RaceResultGUI
 
-//-----------------------------------------------------------------------------
-/** Destructor. */
-RaceResultGUI::~RaceResultGUI()
-{
-}   // ~Racegui
-
-//-----------------------------------------------------------------------------
-void RaceResultGUI::loadedFromFile()
-{
-}   // loadedFromFile
-
 //-----------------------------------------------------------------------------
 /** Besides calling init in the base class this makes all buttons of this 
  *  screen invisible. The buttons will only displayed once the animation is 
@@ -221,20 +210,6 @@ void RaceResultGUI::eventCallback(GUIEngine::Widget* widget,
     }
 }   // eventCallback
 
-//-----------------------------------------------------------------------------
-/** This function is called when one of the player presses 'fire'. The next
- *  phase of the animation will be displayed. E.g. 
- *  in a GP: pressing fire while/after showing the latest race result will
- *           start the animation for the current GP result
- *  in a normal race: when pressing fire while an animation is played, 
- *           start the menu showing 'rerun, new race, back to main' etc.
- */
-void RaceResultGUI::nextPhase()
-{
-    // This will trigger the next phase in the next render call.
-    // FIXME ... work in progress m_timer = 9999;
-}   // nextPhase
-
 //-----------------------------------------------------------------------------
 /** This determines the layout, i.e. the size of all columns, font size etc.
  */
@@ -355,6 +330,33 @@ void RaceResultGUI::determineTableLayout()
 }   // determineTableLayout
 
 //-----------------------------------------------------------------------------
+/** This function is called when one of the player presses 'fire'. The next
+ *  phase of the animation will be displayed. E.g. 
+ *  in a GP: pressing fire while/after showing the latest race result will
+ *           start the animation for the current GP result
+ *  in a normal race: when pressing fire while an animation is played, 
+ *           start the menu showing 'rerun, new race, back to main' etc.
+ */
+void RaceResultGUI::nextPhase()
+{
+    // This will trigger the next phase in the next render call.
+    m_timer = 9999;
+}   // nextPhase
+
+//-----------------------------------------------------------------------------
+/** If escape is pressed, don't do the default option (close the screen), but
+ *  advance to the next animation phase.
+ */
+bool RaceResultGUI::onEscapePressed()
+{
+    nextPhase();
+    return false;   // indicates 'do not close'
+}   // onEscapePressed
+
+//-----------------------------------------------------------------------------
+/** Called once a frame, this now triggers the rendering of the actual
+ *  race result gui.
+ */
 void RaceResultGUI::onUpdate(float dt, irr::video::IVideoDriver*)
 {
     renderGlobal(dt);
diff --git a/src/states_screens/race_result_gui.hpp b/src/states_screens/race_result_gui.hpp
index c197c6e18..b3270ce30 100644
--- a/src/states_screens/race_result_gui.hpp
+++ b/src/states_screens/race_result_gui.hpp
@@ -156,19 +156,14 @@ private:
 public:
 
                  RaceResultGUI();
-    virtual     ~RaceResultGUI();
     virtual void renderGlobal(float dt);
 
     /** \brief Implement callback from parent class GUIEngine::Screen */
-    virtual void loadedFromFile();
+    virtual void loadedFromFile() {};
 
-    /** \brief implement callback from parent class GUIEngine::Screen */
-    void init();
-
-    /** \brief implement callback from parent class GUIEngine::Screen */
-    void tearDown();
-    
-    /** \brief implement callback from parent class GUIEngine::Screen */
+    virtual void init();
+    virtual void tearDown();    
+    virtual bool onEscapePressed();
     void eventCallback(GUIEngine::Widget* widget, const std::string& name, 
                        const int playerID);