From: Thomas Geymayer <tomgey@gmail.com>
Date: Sun, 23 Mar 2014 16:01:02 +0000 (+0100)
Subject: Revert "CanvasGroup: return empty bounding box without children."
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=db8b60ec49ac1e961421fe1ada483ba0f2bea402;p=simgear.git

Revert "CanvasGroup: return empty bounding box without children."

Returning somehow valid data in case it is used unchecked was
probably not a good idea. Better fix the code using it^^

This fixes #1421.

This reverts commit 3525fff8d079538cb36c506b01fdb10128f7c187.
---

diff --git a/simgear/canvas/elements/CanvasGroup.cxx b/simgear/canvas/elements/CanvasGroup.cxx
index 3e284f12..51523f42 100644
--- a/simgear/canvas/elements/CanvasGroup.cxx
+++ b/simgear/canvas/elements/CanvasGroup.cxx
@@ -220,10 +220,7 @@ namespace canvas
   //----------------------------------------------------------------------------
   osg::BoundingBox Group::getTransformedBounds(const osg::Matrix& m) const
   {
-    // Default to a bounding box with size 0x0 and the origin defined by the
-    // matrix.
-    osg::BoundingBox bb(m.getTrans(), m.getTrans());
-
+    osg::BoundingBox bb;
     if( !_transform.valid() )
     {
       warnTransformExpired("getTransformedBounds");