[Haiku-commits] r31202 - haiku/trunk/src/apps/debugger

[stippi at mail.berlios.de]

Author: stippi
Date: 2009-06-23 14:20:14 +0200 (Tue, 23 Jun 2009)
New Revision: 31202
ViewCVS: http://svn.berlios.de/viewcvs/haiku?rev=31202&view=rev

Modified:
   haiku/trunk/src/apps/debugger/Debugger.cpp
Log:
Added NOTE on how to handle QuitRequested in the application and why using
the base class version is fine for now.


Modified: haiku/trunk/src/apps/debugger/Debugger.cpp
===================================================================
--- haiku/trunk/src/apps/debugger/Debugger.cpp	2009-06-23 12:14:05 UTC (rev 31201)
+++ haiku/trunk/src/apps/debugger/Debugger.cpp	2009-06-23 12:20:14 UTC (rev 31202)
@@ -271,7 +271,16 @@
 	{
 		// TODO:...
 //		return true;
-return BApplication::QuitRequested();
+		// NOTE: The default implementation will just ask all windows'
+		// QuitRequested() hooks. This in turn will ask the TeamWindows.
+		// For now, this is what we want. If we have more windows later,
+		// like the global TeamsWindow, then we want to just ask the
+		// TeamDebuggers, the TeamsWindow should of course not go away already
+		// if one or more TeamDebuggers want to stay later. There are multiple
+		// ways how to do this. For examaple, TeamDebugger could get a
+		// QuitReqested() hook or the TeamsWindow and other global windows
+		// could always return false in their QuitRequested().
+		return BApplication::QuitRequested();
 	}
 
 private: