From 6a1af7f74fff2f0490f02f7e7727b9fde01a2e35 Mon Sep 17 00:00:00 2001 From: alberth Date: Sat, 1 Aug 2009 15:49:18 +0000 Subject: (svn r17024) -Codechange: Allow InitNested to be done in two parts. --- src/window.cpp | 66 +++++++++++++++++++++++++++++++++++++++++--------------- src/window_gui.h | 4 +++- 2 files changed, 52 insertions(+), 18 deletions(-) (limited to 'src') diff --git a/src/window.cpp b/src/window.cpp index 4d83af2b3..6c202f489 100644 --- a/src/window.cpp +++ b/src/window.cpp @@ -870,21 +870,22 @@ static void AssignWidgetToWindow(Window *w, const Widget *widget) /** * Initializes the data (except the position and initial size) of a new Window. * @param cls Class of the window, used for identification and grouping. @see WindowClass - * @param *widget Pointer to the widget array. @see Widget - * @param nested_root Root of the nested widget tree. + * @param *widget Pointer to the widget array, it is \c NULL when nested widgets are used. @see Widget * @param window_number Number being assigned to the new window - * @param biggest_index Biggest index of nested widgets in the \a nested_root tree. * @return Window pointer of the newly created window + * @pre If nested widgets are used (\a widget is \c NULL), #nested_root and #nested_array_size must be initialized. + * In addition, #nested_array is either \c NULL, or already initialized. */ -void Window::InitializeData(WindowClass cls, const Widget *widget, NWidgetBase *nested_root, int window_number, int biggest_index) +void Window::InitializeData(WindowClass cls, const Widget *widget, int window_number) { /* If available, initialize nested widget tree. */ - if (nested_root != NULL) { - this->nested_root = nested_root; - /* Setup nested_array pointers into the tree. */ - this->nested_array_size = (uint)(biggest_index + 1); - this->nested_array = CallocT(this->nested_array_size); - this->nested_root->SetupSmallestSize(this, true); + if (widget == NULL) { + if (this->nested_array == NULL) { + this->nested_array = CallocT(this->nested_array_size); + this->nested_root->SetupSmallestSize(this, true); + } else { + this->nested_root->SetupSmallestSize(this, false); + } /* Initialize to smallest size. */ this->nested_root->AssignSizePosition(ST_SMALLEST, 0, 0, this->nested_root->smallest_x, this->nested_root->smallest_y, false, false, false); } @@ -1054,7 +1055,8 @@ void Window::FindWindowPlacementAndResize(const WindowDesc *desc) */ Window::Window(int x, int y, int width, int height, WindowClass cls, const Widget *widget) { - this->InitializeData(cls, widget, NULL, 0, -1); + assert(widget != NULL); // Constructor is used only for old-style widget array windows, they must always provide an old-style widget array. + this->InitializeData(cls, widget, 0); this->InitializePositionSize(x, y, width, height); } @@ -1284,28 +1286,58 @@ static Point LocalGetWindowPlacement(const WindowDesc *desc, int16 sm_width, int */ Window::Window(const WindowDesc *desc, WindowNumber window_number) { - this->InitializeData(desc->cls, desc->GetWidgets(), NULL, window_number, -1); + this->InitializeData(desc->cls, desc->GetWidgets(), window_number); this->desc_flags = desc->flags; Point pt = LocalGetWindowPlacement(desc, desc->minimum_width, desc->minimum_height, window_number); this->InitializePositionSize(pt.x, pt.y, desc->minimum_width, desc->minimum_height); } /** - * Perform initialization of the #Window with nested widgets, to allow use. - * @param desc Window description. - * @param window_number Number of the new window. + * Perform the first part of the initialization of a nested widget tree. + * Construct a nested widget tree in #nested_root, and optionally fill the #nested_array array to provide quick access to the uninitialized widgets. + * This is mainly useful for setting very basic properties. + * @param desc Window description. + * @param fill_nested Fill the #nested_array (enabling is expensive!). + * @note Filling the nested array requires an additional traversal through the nested widget tree, and is best performed by #FinishInitNested rather than here. */ -void Window::InitNested(const WindowDesc *desc, WindowNumber window_number) +void Window::CreateNestedTree(const WindowDesc *desc, bool fill_nested) { int biggest_index = -1; NWidgetBase *nested_root = MakeNWidgets(desc->nwid_parts, desc->nwid_length, &biggest_index); - this->InitializeData(desc->cls, NULL, nested_root, window_number, biggest_index); + this->nested_root = nested_root; + this->nested_array_size = (uint)(biggest_index + 1); + + if (fill_nested) { + this->nested_array = CallocT(this->nested_array_size); + this->nested_root->FillNestedArray(this->nested_array, this->nested_array_size); + } +} + +/** + * Perform the second part of the initialization of a nested widget tree. + * @param desc Window description. + * @param window_number Number of the new window. + */ +void Window::FinishInitNested(const WindowDesc *desc, WindowNumber window_number) +{ + this->InitializeData(desc->cls, NULL, window_number); this->desc_flags = desc->flags; Point pt = LocalGetWindowPlacement(desc, this->nested_root->smallest_x, this->nested_root->smallest_y, window_number); this->InitializePositionSize(pt.x, pt.y, this->nested_root->smallest_x, this->nested_root->smallest_y); this->FindWindowPlacementAndResize(desc->default_width, desc->default_height); } +/** + * Perform complete initialization of the #Window with nested widgets, to allow use. + * @param desc Window description. + * @param window_number Number of the new window. + */ +void Window::InitNested(const WindowDesc *desc, WindowNumber window_number) +{ + this->CreateNestedTree(desc, false); + this->FinishInitNested(desc, window_number); +} + /** Empty constructor, initialization has been moved to #InitNested() called from the constructor of the derived class. */ Window::Window() { diff --git a/src/window_gui.h b/src/window_gui.h index f074093ef..d5ad1ca6d 100644 --- a/src/window_gui.h +++ b/src/window_gui.h @@ -220,7 +220,7 @@ struct Window : ZeroedMemoryAllocator { }; protected: - void InitializeData(WindowClass cls, const Widget *widget, NWidgetBase *nwid, int window_number, int biggest_index); + void InitializeData(WindowClass cls, const Widget *widget, int window_number); void InitializePositionSize(int x, int y, int min_width, int min_height); void FindWindowPlacementAndResize(int def_width, int def_height); void FindWindowPlacementAndResize(const WindowDesc *desc); @@ -268,6 +268,8 @@ public: Window *z_back; ///< The window behind us in z-order. void InitNested(const WindowDesc *desc, WindowNumber number = 0); + void CreateNestedTree(const WindowDesc *desc, bool fill_nested = true); + void FinishInitNested(const WindowDesc *desc, WindowNumber window_number); /** * Sets the enabled/disabled status of a widget. -- cgit v1.2.3-54-g00ecf