From eb7d55e218fc89ec1b5582ed5d0f8cdede29cae5 Mon Sep 17 00:00:00 2001 From: Albrecht Schlosser Date: Mon, 27 Jul 2015 18:13:46 +0000 Subject: Document Fl_Window::show() calling Fl_Group::current(0). Also fix indenting in src/Fl_win32.cxx. git-svn-id: file:///fltk/svn/fltk/branches/branch-1.3@10820 ea41ed52-d2ee-0310-a9c1-e6b18d33e121 --- FL/Fl_Window.H | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-) (limited to 'FL') diff --git a/FL/Fl_Window.H b/FL/Fl_Window.H index c703d7154..f84ab1c9a 100644 --- a/FL/Fl_Window.H +++ b/FL/Fl_Window.H @@ -491,11 +491,22 @@ public: top. This is really convenient because your program can call show() at any time, even if the window is already up. It also means that show() serves the purpose of raise() in other toolkits. - + Fl_Window::show(int argc, char **argv) is used for top-level windows and allows standard arguments to be parsed from the command-line. - + + \note For some obscure reasons Fl_Window::show() resets the current + group by calling Fl_Group::current(0). The comments in the code + say "get rid of very common user bug: forgot end()". Although + this is true it may have unwanted side effects if you show() an + unrelated window (maybe for an error message or warning) while + building a window or any other group widget. + + \todo Check if we can remove resetting the current group in a later + FLTK version (after 1.3.x). This may break "already broken" programs + though if they rely on this "feature". + \see Fl_Window::show(int argc, char **argv) */ virtual void show(); -- cgit v1.2.3