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
This commit is contained in:
Albrecht Schlosser 2015-07-27 18:13:46 +00:00
parent a165f1ca3c
commit eb7d55e218
2 changed files with 15 additions and 4 deletions

View File

@ -491,11 +491,22 @@ public:
top. This is really convenient because your program can call show() 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 at any time, even if the window is already up. It also means that
show() serves the purpose of raise() in other toolkits. show() serves the purpose of raise() in other toolkits.
Fl_Window::show(int argc, char **argv) is used for top-level Fl_Window::show(int argc, char **argv) is used for top-level
windows and allows standard arguments to be parsed from the windows and allows standard arguments to be parsed from the
command-line. 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) \see Fl_Window::show(int argc, char **argv)
*/ */
virtual void show(); virtual void show();

View File

@ -2541,8 +2541,8 @@ void Fl_Window::show() {
//ShowWindow(i->xid,fl_capture?SW_SHOWNOACTIVATE:SW_RESTORE); //ShowWindow(i->xid,fl_capture?SW_SHOWNOACTIVATE:SW_RESTORE);
} }
#ifdef USE_PRINT_BUTTON #ifdef USE_PRINT_BUTTON
void preparePrintFront(void); void preparePrintFront(void);
preparePrintFront(); preparePrintFront();
#endif #endif
} }