Update OpenVMS compile support

[wxWidgets.git] / src / common / sizer.cpp

diff --git a/src/common/sizer.cpp b/src/common/sizer.cpp
index 071ee210399dd61bff052261de581b394e3ba23e..beb46796bc4b0a8cb479376d27a814cb06893709 100644 (file)
--- a/src/common/sizer.cpp
+++ b/src/common/sizer.cpp
@@ -17,7 +17,6 @@
     #pragma hdrstop
 #endif
 
     #pragma hdrstop
 #endif
 

-#include "wx/display.h"

 #include "wx/sizer.h"
 #include "wx/private/flagscheck.h"
 
 #include "wx/sizer.h"
 #include "wx/private/flagscheck.h"
 


@@ -32,6 +31,7 @@
     #include "wx/toplevel.h"
 #endif // WX_PRECOMP
 
     #include "wx/toplevel.h"
 #endif // WX_PRECOMP
 

+#include "wx/display.h"

 #include "wx/listimpl.cpp"
 
 
 #include "wx/listimpl.cpp"
 
 


@@ -1631,11 +1631,9 @@ wxSize wxFlexGridSizer::CalcMin()
     m_rowHeights.assign(nrows, -1);
     m_colWidths.assign(ncols, -1);
 
     m_rowHeights.assign(nrows, -1);
     m_colWidths.assign(ncols, -1);
 

-    // n is the index of the item in left-to-right top-to-bottom order
-    size_t n = 0;

     for ( wxSizerItemList::iterator i = m_children.begin();
           i != m_children.end();
     for ( wxSizerItemList::iterator i = m_children.begin();
           i != m_children.end();

-          ++i, ++n )
+          ++i)

     {
         wxSizerItem * const item = *i;
         if ( item->IsShown() )
     {
         wxSizerItem * const item = *i;
         if ( item->IsShown() )


@@ -1789,21 +1787,21 @@ void wxFlexGridSizer::AdjustForGrowables(const wxSize& sz)
         {
             const int col = n % ncols;
             didAdjustMinSize |= (*i)->InformFirstDirection(wxHORIZONTAL, m_colWidths[col], sz.y - m_calculatedMinSize.y);
         {
             const int col = n % ncols;
             didAdjustMinSize |= (*i)->InformFirstDirection(wxHORIZONTAL, m_colWidths[col], sz.y - m_calculatedMinSize.y);

-    }
+        }

 
         // Only redo if info was actually used
         if( didAdjustMinSize )
 
         // Only redo if info was actually used
         if( didAdjustMinSize )

-    {
-        DoAdjustForGrowables
-        (
-            sz.x - m_calculatedMinSize.x,
-            m_growableCols,
-            m_colWidths,
-            m_growMode == wxFLEX_GROWMODE_SPECIFIED ? &m_growableColsProportions
-                                                    : NULL
-        );
+        {
+            DoAdjustForGrowables
+            (
+                sz.x - m_calculatedMinSize.x,
+                m_growableCols,
+                m_colWidths,
+                m_growMode == wxFLEX_GROWMODE_SPECIFIED ? &m_growableColsProportions
+                                                        : NULL
+            );
+        }

     }
     }

-}

 
     if ( (m_flexDirection & wxVERTICAL) || (m_growMode != wxFLEX_GROWMODE_NONE) )
     {
 
     if ( (m_flexDirection & wxVERTICAL) || (m_growMode != wxFLEX_GROWMODE_NONE) )
     {


@@ -1939,15 +1937,22 @@ void wxBoxSizer::RecalcSizes()
 
         // adjust the size in the major direction using the proportion
         wxCoord majorSize = GetSizeInMajorDir(sizeThis);
 
         // adjust the size in the major direction using the proportion
         wxCoord majorSize = GetSizeInMajorDir(sizeThis);

-        const int propItem = item->GetProportion();
-        if ( propItem )
+
+        // if there is not enough space, don't try to distribute negative space
+        // among the children, this would result in overlapping windows which
+        // we don't want
+        if ( delta > 0 )

         {
         {

-            const int deltaItem = (delta * propItem) / totalProportion;
+            const int propItem = item->GetProportion();
+            if ( propItem )
+            {
+                const int deltaItem = (delta * propItem) / totalProportion;

 
 

-            majorSize += deltaItem;
+                majorSize += deltaItem;

 
 

-            delta -= deltaItem;
-            totalProportion -= propItem;
+                delta -= deltaItem;
+                totalProportion -= propItem;
+            }

         }
 
 
         }