Window::getChild should not be recursive

Create issue
Issue #237 resolved
Paul Turner created an issue

Between 0.5.x and 0.6.0 the Window::getChild(const CEGUI::String&) function was modified to be recursive against the original intention.

The intention of the Window::getChild functions is to scan the immediate children, not to scan all children recursively which in many cases will be prohibitively expensive.

Reproducibility: always

Additional information: There was already a Window::getChildRecursive function added for use with Window ID codes, so the correct solution would have been to add a Window::getChildRecursive function overload that accepts a CEGUI::String rather than butchering the original non-recursive function.

The solution here should be to revert the old behaviour of Window::getChild and introduce the new Window::getChildRecursive overload.

Comments (1)

  1. Log in to comment