[haiku-commits] Re: haiku: hrev45628 - src/apps/debugger/user_interface/gui/inspector_window

  • From: Stephan Aßmus <superstippi@xxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Mon, 06 May 2013 08:33:58 +0200

Hi,

On 05.05.2013 19:34, anevilyak@xxxxxxxxx wrote:
diff --git 
a/src/apps/debugger/user_interface/gui/inspector_window/InspectorWindow.cpp 
b/src/apps/debugger/user_interface/gui/inspector_window/InspectorWindow.cpp
index 5dbb108..ed7ec9a 100644
--- a/src/apps/debugger/user_interface/gui/inspector_window/InspectorWindow.cpp
+++ b/src/apps/debugger/user_interface/gui/inspector_window/InspectorWindow.cpp
@@ -125,7 +125,7 @@ InspectorWindow::_Init()
        textMenu->AddItem(item);

        BLayoutBuilder::Group<>(this, B_VERTICAL)
-               .SetInsets(4.0f, 4.0f, 4.0f, 4.0f)
+               .SetInsets(B_USE_DEFAULT_SPACING)
                .AddGroup(B_HORIZONTAL, 4.0f)

Are you sure the AddGroup() call below shouldn't be changed from 4.0 to B_USE_DEFAULT_SPACING? (I think equivalent to removing the 4.0 argument.)

I spotted this in the earlier commits as well.

Best regards,
-Stephan



Other related posts: