diff libgui/graphics/QtHandlesUtils.cc @ 31648:29d734430e5f stable

maint: Re-indent code after switch to using namespace macros. * BaseControl.cc, BaseControl.h, ButtonControl.cc, ButtonControl.h, ButtonGroup.cc, ButtonGroup.h, Canvas.cc, Canvas.h, CheckBoxControl.cc, CheckBoxControl.h, Container.cc, Container.h, ContextMenu.cc, ContextMenu.h, EditControl.cc, EditControl.h, Figure.cc, Figure.h, FigureWindow.cc, FigureWindow.h, GLCanvas.cc, GLCanvas.h, GenericEventNotify.h, KeyMap.cc, KeyMap.h, ListBoxControl.cc, ListBoxControl.h, Logger.cc, Logger.h, Menu.cc, Menu.h, MenuContainer.h, Object.cc, Object.h, ObjectProxy.cc, ObjectProxy.h, Panel.cc, Panel.h, PopupMenuControl.cc, PopupMenuControl.h, PushButtonControl.cc, PushButtonControl.h, PushTool.cc, PushTool.h, QtHandlesUtils.cc, QtHandlesUtils.h, RadioButtonControl.cc, RadioButtonControl.h, SliderControl.cc, SliderControl.h, Table.cc, Table.h, TextControl.cc, TextControl.h, TextEdit.cc, TextEdit.h, ToggleButtonControl.cc, ToggleButtonControl.h, ToggleTool.cc, ToggleTool.h, ToolBar.cc, ToolBar.h, ToolBarButton.cc, ToolBarButton.h, annotation-dialog.cc, annotation-dialog.h, gl-select.cc, gl-select.h, qopengl-functions.h, qt-graphics-toolkit.cc, qt-graphics-toolkit.h, module.mk, QTerminal.h, color-picker.cc, color-picker.h, command-widget.cc, command-widget.h, community-news.cc, community-news.h, dialog.cc, dialog.h, documentation-bookmarks.cc, documentation-bookmarks.h, documentation-dock-widget.cc, documentation-dock-widget.h, documentation.cc, documentation.h, dw-main-window.cc, dw-main-window.h, external-editor-interface.cc, external-editor-interface.h, files-dock-widget.cc, files-dock-widget.h, find-files-dialog.cc, find-files-dialog.h, find-files-model.cc, find-files-model.h, graphics-init.cc, graphics-init.h, gui-settings.cc, gui-settings.h, gui-utils.cc, gui-utils.h, history-dock-widget.cc, history-dock-widget.h, interpreter-qobject.cc, interpreter-qobject.h, led-indicator.cc, led-indicator.h, file-editor-interface.h, file-editor-tab.cc, file-editor-tab.h, file-editor.cc, file-editor.h, find-dialog.cc, find-dialog.h, marker.cc, marker.h, octave-qscintilla.cc, octave-qscintilla.h, octave-txt-lexer.cc, octave-txt-lexer.h, main-window.cc, main-window.h, news-reader.cc, news-reader.h, octave-dock-widget.cc, octave-dock-widget.h, octave-qobject.cc, octave-qobject.h, qt-application.cc, qt-application.h, qt-interpreter-events.cc, qt-interpreter-events.h, qt-utils.h, release-notes.cc, release-notes.h, resource-manager.cc, resource-manager.h, set-path-dialog.cc, set-path-dialog.h, set-path-model.cc, set-path-model.h, settings-dialog.cc, settings-dialog.h, shortcut-manager.cc, shortcut-manager.h, tab-bar.cc, tab-bar.h, terminal-dock-widget.cc, terminal-dock-widget.h, variable-editor-model.cc, variable-editor-model.h, variable-editor.cc, variable-editor.h, welcome-wizard.cc, welcome-wizard.h, workspace-model.cc, workspace-model.h, workspace-view.cc, workspace-view.h: Re-indent code after switch to using namespace macros.
author John W. Eaton <jwe@octave.org>
date Tue, 06 Dec 2022 14:53:00 -0500
parents c6d54dd31a7e
children 597f3ee61a48
line wrap: on
line diff
--- a/libgui/graphics/QtHandlesUtils.cc	Tue Dec 06 14:37:51 2022 -0500
+++ b/libgui/graphics/QtHandlesUtils.cc	Tue Dec 06 14:53:00 2022 -0500
@@ -46,376 +46,376 @@
 
 OCTAVE_BEGIN_NAMESPACE(octave)
 
-  namespace Utils
-  {
+namespace Utils
+{
 
-    QString
-    fromStdString (const std::string& s)
-    {
-      return QString::fromUtf8 (s.c_str ());
-    }
+  QString
+  fromStdString (const std::string& s)
+  {
+    return QString::fromUtf8 (s.c_str ());
+  }
 
-    std::string
-    toStdString (const QString& s)
-    {
-      return std::string (s.toUtf8 ().data ());
-    }
+  std::string
+  toStdString (const QString& s)
+  {
+    return std::string (s.toUtf8 ().data ());
+  }
 
-    QStringList
-    fromStringVector (const string_vector& v)
-    {
-      QStringList l;
-      octave_idx_type n = v.numel ();
+  QStringList
+  fromStringVector (const string_vector& v)
+  {
+    QStringList l;
+    octave_idx_type n = v.numel ();
 
-      for (octave_idx_type i = 0; i < n; i++)
-        l << fromStdString (v[i]);
+    for (octave_idx_type i = 0; i < n; i++)
+      l << fromStdString (v[i]);
 
-      return l;
-    }
+    return l;
+  }
 
-    string_vector
-    toStringVector (const QStringList& l)
-    {
-      string_vector v (l.length ());
-      int i = 0;
+  string_vector
+  toStringVector (const QStringList& l)
+  {
+    string_vector v (l.length ());
+    int i = 0;
 
-      for (const auto& s : l)
-        v[i++] = toStdString (s);
+    for (const auto& s : l)
+      v[i++] = toStdString (s);
 
-      return v;
-    }
+    return v;
+  }
 
-    Cell toCellString (const QStringList& l)
-    {
-      QStringList tmp = l;
+  Cell toCellString (const QStringList& l)
+  {
+    QStringList tmp = l;
 
-      // don't get any empty lines from end of the list
-      while ((tmp.length () > 0) && tmp.last ().isEmpty ())
-        {
-          tmp.removeLast ();
-        }
-      // no strings converts to a 1x1 cell with empty string
-      if (tmp.isEmpty ())
-        tmp += "";
+    // don't get any empty lines from end of the list
+    while ((tmp.length () > 0) && tmp.last ().isEmpty ())
+      {
+        tmp.removeLast ();
+      }
+    // no strings converts to a 1x1 cell with empty string
+    if (tmp.isEmpty ())
+      tmp += "";
 
-      Cell v (toStringVector (tmp));
-      return v;
-    }
+    Cell v (toStringVector (tmp));
+    return v;
+  }
 
-    template <typename T>
-    QFont
-    computeFont (const typename T::properties& props, int height)
-    {
-      QFont f (fromStdString (props.get_fontname ()));
-
-      static std::map<std::string, QFont::Weight> weightMap;
-      static std::map<std::string, QFont::Style> angleMap;
-      static bool mapsInitialized = false;
+  template <typename T>
+  QFont
+  computeFont (const typename T::properties& props, int height)
+  {
+    QFont f (fromStdString (props.get_fontname ()));
 
-      if (! mapsInitialized)
-        {
-          weightMap["normal"] = QFont::Normal;
-          weightMap["bold"] = QFont::Bold;
+    static std::map<std::string, QFont::Weight> weightMap;
+    static std::map<std::string, QFont::Style> angleMap;
+    static bool mapsInitialized = false;
 
-          angleMap["normal"] = QFont::StyleNormal;
-          angleMap["italic"] = QFont::StyleItalic;
-          angleMap["oblique"] = QFont::StyleOblique;
-
-          mapsInitialized = true;
-        }
+    if (! mapsInitialized)
+      {
+        weightMap["normal"] = QFont::Normal;
+        weightMap["bold"] = QFont::Bold;
 
-      f.setPointSizeF (props.get___fontsize_points__ (height));
-      f.setWeight (weightMap[props.get_fontweight ()]);
-      f.setStyle (angleMap[props.get_fontangle ()]);
+        angleMap["normal"] = QFont::StyleNormal;
+        angleMap["italic"] = QFont::StyleItalic;
+        angleMap["oblique"] = QFont::StyleOblique;
 
-      return f;
-    }
+        mapsInitialized = true;
+      }
 
-    template QFont computeFont<uicontrol> (const uicontrol::properties& props,
-                                           int height);
-
-    template QFont computeFont<uipanel> (const uipanel::properties& props,
-                                         int height);
+    f.setPointSizeF (props.get___fontsize_points__ (height));
+    f.setWeight (weightMap[props.get_fontweight ()]);
+    f.setStyle (angleMap[props.get_fontangle ()]);
 
-    template QFont computeFont<uibuttongroup> (const uibuttongroup::properties&
-        props,
-        int height);
+    return f;
+  }
 
-    template QFont computeFont<uitable> (const uitable::properties& props,
+  template QFont computeFont<uicontrol> (const uicontrol::properties& props,
                                          int height);
 
-    QColor
-    fromRgb (const Matrix& rgb)
-    {
-      QColor c;
+  template QFont computeFont<uipanel> (const uipanel::properties& props,
+                                       int height);
 
-      if (rgb.numel () == 3)
-        c.setRgbF (rgb(0), rgb(1), rgb(2));
+  template QFont computeFont<uibuttongroup> (const uibuttongroup::properties&
+                                             props,
+                                             int height);
 
-      return c;
-    }
+  template QFont computeFont<uitable> (const uitable::properties& props,
+                                       int height);
 
-    Matrix
-    toRgb (const QColor& c)
-    {
-      Matrix rgb (1, 3);
-      double *rgbData = rgb.fortran_vec ();
+  QColor
+  fromRgb (const Matrix& rgb)
+  {
+    QColor c;
 
-      // qreal is a typedef for double except for ARM CPU architectures
-      // where it is a typedef for float (Bug #44970).
-      qreal tmp[3];
-      c.getRgbF (tmp, tmp+1, tmp+2);
-      rgbData[0] = tmp[0]; rgbData[1] = tmp[1]; rgbData[2] = tmp[2];
+    if (rgb.numel () == 3)
+      c.setRgbF (rgb(0), rgb(1), rgb(2));
 
-      return rgb;
-    }
+    return c;
+  }
 
-    std::string
-    figureSelectionType (QMouseEvent *event, bool isDoubleClick)
-    {
-      if (isDoubleClick)
-        return "open";
-      else
-        {
-          Qt::MouseButtons buttons = event->buttons ();
-          Qt::KeyboardModifiers mods = event->modifiers ();
+  Matrix
+  toRgb (const QColor& c)
+  {
+    Matrix rgb (1, 3);
+    double *rgbData = rgb.fortran_vec ();
+
+    // qreal is a typedef for double except for ARM CPU architectures
+    // where it is a typedef for float (Bug #44970).
+    qreal tmp[3];
+    c.getRgbF (tmp, tmp+1, tmp+2);
+    rgbData[0] = tmp[0]; rgbData[1] = tmp[1]; rgbData[2] = tmp[2];
 
-          if (mods == Qt::NoModifier)
-            {
-              if (buttons == Qt::LeftButton)
-                return "normal";
-              else if (buttons == Qt::RightButton)
-                return "alt";
-              else if (buttons == Qt::MiddleButton
-                       || buttons == (Qt::LeftButton | Qt::RightButton))
-                return "extend";
-            }
-          else if (buttons == Qt::LeftButton)
-            {
-              if (mods == Qt::ShiftModifier)
-                return "extend";
-              else if (mods == Qt::ControlModifier)
-                return "alt";
-            }
-        }
+    return rgb;
+  }
 
-      return "normal";
-    }
+  std::string
+  figureSelectionType (QMouseEvent *event, bool isDoubleClick)
+  {
+    if (isDoubleClick)
+      return "open";
+    else
+      {
+        Qt::MouseButtons buttons = event->buttons ();
+        Qt::KeyboardModifiers mods = event->modifiers ();
 
-    /*
-       Two figureCurrentPoint() routines are required:
-       1) Used for QMouseEvents where cursor position data is in callback from Qt.
-       2) Used for QKeyEvents where cursor position must be determined.
-    */
-    Matrix
-    figureCurrentPoint (const graphics_object& fig, QMouseEvent *event)
-    {
-      Object *tkFig = qt_graphics_toolkit::toolkitObject (fig);
-
-      if (tkFig)
-        {
-          Container *c = tkFig->innerContainer ();
+        if (mods == Qt::NoModifier)
+          {
+            if (buttons == Qt::LeftButton)
+              return "normal";
+            else if (buttons == Qt::RightButton)
+              return "alt";
+            else if (buttons == Qt::MiddleButton
+                     || buttons == (Qt::LeftButton | Qt::RightButton))
+              return "extend";
+          }
+        else if (buttons == Qt::LeftButton)
+          {
+            if (mods == Qt::ShiftModifier)
+              return "extend";
+            else if (mods == Qt::ControlModifier)
+              return "alt";
+          }
+      }
 
-          if (c)
-            {
-              QPoint qp = c->mapFromGlobal (event->globalPos ());
+    return "normal";
+  }
 
-              return tkFig->properties<figure> ().map_from_boundingbox (qp.x (),
-                     qp.y ());
-            }
-        }
+  /*
+    Two figureCurrentPoint() routines are required:
+    1) Used for QMouseEvents where cursor position data is in callback from Qt.
+    2) Used for QKeyEvents where cursor position must be determined.
+  */
+  Matrix
+  figureCurrentPoint (const graphics_object& fig, QMouseEvent *event)
+  {
+    Object *tkFig = qt_graphics_toolkit::toolkitObject (fig);
 
-      return Matrix (1, 2, 0.0);
-    }
-
-    Matrix
-    figureCurrentPoint (const graphics_object& fig)
-    {
-      Object *tkFig = qt_graphics_toolkit::toolkitObject (fig);
+    if (tkFig)
+      {
+        Container *c = tkFig->innerContainer ();
 
-      if (tkFig)
-        {
-          Container *c = tkFig->innerContainer ();
+        if (c)
+          {
+            QPoint qp = c->mapFromGlobal (event->globalPos ());
 
-          if (c)
-            {
-              // FIXME: QCursor::pos() may give inaccurate results with
-              //        asynchronous window systems like X11 over ssh.
-              QPoint qp = c->mapFromGlobal (QCursor::pos ());
+            return tkFig->properties<figure> ().map_from_boundingbox (qp.x (),
+                                                                      qp.y ());
+          }
+      }
 
-              return tkFig->properties<figure> ().map_from_boundingbox (qp.x (),
-                     qp.y ());
-            }
-        }
+    return Matrix (1, 2, 0.0);
+  }
+
+  Matrix
+  figureCurrentPoint (const graphics_object& fig)
+  {
+    Object *tkFig = qt_graphics_toolkit::toolkitObject (fig);
 
-      return Matrix (1, 2, 0.0);
-    }
+    if (tkFig)
+      {
+        Container *c = tkFig->innerContainer ();
 
-    Qt::Alignment
-    fromHVAlign (const std::string& halign, const std::string& valign)
-    {
-      Qt::Alignment flags;
+        if (c)
+          {
+            // FIXME: QCursor::pos() may give inaccurate results with
+            //        asynchronous window systems like X11 over ssh.
+            QPoint qp = c->mapFromGlobal (QCursor::pos ());
 
-      if (octave::string::strcmpi (halign, "left"))
-        flags |= Qt::AlignLeft;
-      else if (octave::string::strcmpi (halign, "center"))
-        flags |= Qt::AlignHCenter;
-      else if (octave::string::strcmpi (halign, "right"))
-        flags |= Qt::AlignRight;
-      else
-        flags |= Qt::AlignLeft;
+            return tkFig->properties<figure> ().map_from_boundingbox (qp.x (),
+                                                                      qp.y ());
+          }
+      }
+
+    return Matrix (1, 2, 0.0);
+  }
+
+  Qt::Alignment
+  fromHVAlign (const std::string& halign, const std::string& valign)
+  {
+    Qt::Alignment flags;
 
-      if (octave::string::strcmpi (valign, "middle"))
-        flags |= Qt::AlignVCenter;
-      else if (octave::string::strcmpi (valign, "top"))
-        flags |= Qt::AlignTop;
-      else if (octave::string::strcmpi (valign, "bottom"))
-        flags |= Qt::AlignBottom;
-      else
-        flags |= Qt::AlignVCenter;
+    if (octave::string::strcmpi (halign, "left"))
+      flags |= Qt::AlignLeft;
+    else if (octave::string::strcmpi (halign, "center"))
+      flags |= Qt::AlignHCenter;
+    else if (octave::string::strcmpi (halign, "right"))
+      flags |= Qt::AlignRight;
+    else
+      flags |= Qt::AlignLeft;
 
-      return flags;
-    }
+    if (octave::string::strcmpi (valign, "middle"))
+      flags |= Qt::AlignVCenter;
+    else if (octave::string::strcmpi (valign, "top"))
+      flags |= Qt::AlignTop;
+    else if (octave::string::strcmpi (valign, "bottom"))
+      flags |= Qt::AlignBottom;
+    else
+      flags |= Qt::AlignVCenter;
 
-    QImage
-    makeImageFromCData (const octave_value& v, int width, int height)
-    {
-      dim_vector dv (v.dims ());
+    return flags;
+  }
 
-      if (dv.ndims () == 3 && dv(2) == 3)
-        {
-          int w = qMin (dv(1), static_cast<octave_idx_type> (width));
-          int h = qMin (dv(0), static_cast<octave_idx_type> (height));
+  QImage
+  makeImageFromCData (const octave_value& v, int width, int height)
+  {
+    dim_vector dv (v.dims ());
+
+    if (dv.ndims () == 3 && dv(2) == 3)
+      {
+        int w = qMin (dv(1), static_cast<octave_idx_type> (width));
+        int h = qMin (dv(0), static_cast<octave_idx_type> (height));
 
-          // If size mismatch, take data from center of CDATA and
-          // place in in center of QImage.
-          int x_img_off = (w < width ? (width - w) / 2 : 0);
-          int y_img_off = (h < height ? (height - h) / 2 : 0);
-          int x_cdat_off = (dv(1) > w ? (dv(1) - w) / 2 : 0);
-          int y_cdat_off = (dv(0) > h ? (dv(0) - h) / 2 : 0);
+        // If size mismatch, take data from center of CDATA and
+        // place in in center of QImage.
+        int x_img_off = (w < width ? (width - w) / 2 : 0);
+        int y_img_off = (h < height ? (height - h) / 2 : 0);
+        int x_cdat_off = (dv(1) > w ? (dv(1) - w) / 2 : 0);
+        int y_cdat_off = (dv(0) > h ? (dv(0) - h) / 2 : 0);
 
-          QImage img (width, height, QImage::Format_ARGB32);
-          img.fill (qRgba (0, 0, 0, 0));
+        QImage img (width, height, QImage::Format_ARGB32);
+        img.fill (qRgba (0, 0, 0, 0));
 
-          if (v.is_uint8_type ())
-            {
-              uint8NDArray d = v.uint8_array_value ();
+        if (v.is_uint8_type ())
+          {
+            uint8NDArray d = v.uint8_array_value ();
 
-              for (int i = x_cdat_off; i < w + x_cdat_off; i++)
-                for (int j = y_cdat_off; j < h + y_cdat_off; j++)
-                  {
-                    int r = d(j, i, 0);
-                    int g = d(j, i, 1);
-                    int b = d(j, i, 2);
-                    int a = 255;
+            for (int i = x_cdat_off; i < w + x_cdat_off; i++)
+              for (int j = y_cdat_off; j < h + y_cdat_off; j++)
+                {
+                  int r = d(j, i, 0);
+                  int g = d(j, i, 1);
+                  int b = d(j, i, 2);
+                  int a = 255;
 
-                    img.setPixel (x_img_off + i - x_cdat_off,
-                                  y_img_off + j - y_cdat_off,
-                                  qRgba (r, g, b, a));
-                  }
-            }
-          else if (v.is_single_type ())
-            {
-              FloatNDArray f = v.float_array_value ();
+                  img.setPixel (x_img_off + i - x_cdat_off,
+                                y_img_off + j - y_cdat_off,
+                                qRgba (r, g, b, a));
+                }
+          }
+        else if (v.is_single_type ())
+          {
+            FloatNDArray f = v.float_array_value ();
 
-              for (int i = x_cdat_off; i < w + x_cdat_off; i++)
-                for (int j = y_cdat_off; j < h + y_cdat_off; j++)
-                  {
-                    float r = f(j, i, 0);
-                    float g = f(j, i, 1);
-                    float b = f(j, i, 2);
-                    int a = (octave::math::isnan (r) || octave::math::isnan (g)
-                             || octave::math::isnan (b) ? 0 : 255);
+            for (int i = x_cdat_off; i < w + x_cdat_off; i++)
+              for (int j = y_cdat_off; j < h + y_cdat_off; j++)
+                {
+                  float r = f(j, i, 0);
+                  float g = f(j, i, 1);
+                  float b = f(j, i, 2);
+                  int a = (octave::math::isnan (r) || octave::math::isnan (g)
+                           || octave::math::isnan (b) ? 0 : 255);
 
-                    img.setPixel (x_img_off + i - x_cdat_off,
-                                  y_img_off + j - y_cdat_off,
-                                  qRgba (octave::math::round (r * 255),
-                                         octave::math::round (g * 255),
-                                         octave::math::round (b * 255),
-                                         a));
-                  }
-            }
-          else if (v.isreal ())
-            {
-              NDArray d = v.array_value ();
+                  img.setPixel (x_img_off + i - x_cdat_off,
+                                y_img_off + j - y_cdat_off,
+                                qRgba (octave::math::round (r * 255),
+                                       octave::math::round (g * 255),
+                                       octave::math::round (b * 255),
+                                       a));
+                }
+          }
+        else if (v.isreal ())
+          {
+            NDArray d = v.array_value ();
 
-              for (int i = x_cdat_off; i < w + x_cdat_off; i++)
-                for (int j = y_cdat_off; j < h + y_cdat_off; j++)
-                  {
-                    double r = d(j, i, 0);
-                    double g = d(j, i, 1);
-                    double b = d(j, i, 2);
-                    int a = (octave::math::isnan (r) || octave::math::isnan (g)
-                             || octave::math::isnan (b) ? 0 : 255);
+            for (int i = x_cdat_off; i < w + x_cdat_off; i++)
+              for (int j = y_cdat_off; j < h + y_cdat_off; j++)
+                {
+                  double r = d(j, i, 0);
+                  double g = d(j, i, 1);
+                  double b = d(j, i, 2);
+                  int a = (octave::math::isnan (r) || octave::math::isnan (g)
+                           || octave::math::isnan (b) ? 0 : 255);
 
-                    img.setPixel (x_img_off + i - x_cdat_off,
-                                  y_img_off + j - y_cdat_off,
-                                  qRgba (octave::math::round (r * 255),
-                                         octave::math::round (g * 255),
-                                         octave::math::round (b * 255),
-                                         a));
-                  }
-            }
+                  img.setPixel (x_img_off + i - x_cdat_off,
+                                y_img_off + j - y_cdat_off,
+                                qRgba (octave::math::round (r * 255),
+                                       octave::math::round (g * 255),
+                                       octave::math::round (b * 255),
+                                       a));
+                }
+          }
 
-          return img;
-        }
+        return img;
+      }
 
-      return QImage ();
-    }
+    return QImage ();
+  }
 
-    octave_scalar_map
-    makeKeyEventStruct (QKeyEvent *event)
-    {
-      octave_scalar_map retval;
+  octave_scalar_map
+  makeKeyEventStruct (QKeyEvent *event)
+  {
+    octave_scalar_map retval;
 
-      retval.setfield ("Key", KeyMap::qKeyToKeyString (event->key ()));
-      retval.setfield ("Character", toStdString (event->text ()));
+    retval.setfield ("Key", KeyMap::qKeyToKeyString (event->key ()));
+    retval.setfield ("Character", toStdString (event->text ()));
 
-      std::list<std::string> modList;
-      Qt::KeyboardModifiers mods = event->modifiers ();
+    std::list<std::string> modList;
+    Qt::KeyboardModifiers mods = event->modifiers ();
 
-      if (mods & Qt::ShiftModifier)
-        modList.push_back ("shift");
-      if (mods & Qt::ControlModifier)
+    if (mods & Qt::ShiftModifier)
+      modList.push_back ("shift");
+    if (mods & Qt::ControlModifier)
 #if defined (Q_OS_MAC)
-        modList.push_back ("command");
+      modList.push_back ("command");
 #else
-        modList.push_back ("control");
+    modList.push_back ("control");
 #endif
-      if (mods & Qt::AltModifier)
-        modList.push_back ("alt");
+    if (mods & Qt::AltModifier)
+      modList.push_back ("alt");
 #if defined (Q_OS_MAC)
-      if (mods & Qt::MetaModifier)
-        modList.push_back ("control");
+    if (mods & Qt::MetaModifier)
+      modList.push_back ("control");
 #endif
 
-      retval.setfield ("Modifier", Cell (modList));
-
-      return retval;
-    }
-
-    octave_scalar_map
-    makeScrollEventStruct (QWheelEvent *event)
-    {
-      octave_scalar_map retval;
+    retval.setfield ("Modifier", Cell (modList));
 
-      // We assume a standard mouse with 15 degree steps and Qt returns
-      // 1/8 of a degree.
-#if defined (HAVE_QWHEELEVENT_ANGLEDELTA)
-      int ydelta = -(event->angleDelta().y ());
-#else
-      int ydelta = -(event->delta ());
-#endif
-      retval.setfield ("VerticalScrollCount", octave_value (ydelta / 120));
-
-      // FIXME: Is there any way to access the number of lines a scroll step
-      // should correspond to?
-      retval.setfield ("VerticalScrollAmount", octave_value (3));
-      retval.setfield ("EventName", octave_value ("WindowScrollWheel"));
-
-      return retval;
-    }
-
+    return retval;
   }
 
+  octave_scalar_map
+  makeScrollEventStruct (QWheelEvent *event)
+  {
+    octave_scalar_map retval;
+
+    // We assume a standard mouse with 15 degree steps and Qt returns
+    // 1/8 of a degree.
+#if defined (HAVE_QWHEELEVENT_ANGLEDELTA)
+    int ydelta = -(event->angleDelta().y ());
+#else
+    int ydelta = -(event->delta ());
+#endif
+    retval.setfield ("VerticalScrollCount", octave_value (ydelta / 120));
+
+    // FIXME: Is there any way to access the number of lines a scroll step
+    // should correspond to?
+    retval.setfield ("VerticalScrollAmount", octave_value (3));
+    retval.setfield ("EventName", octave_value ("WindowScrollWheel"));
+
+    return retval;
+  }
+
+}
+
 OCTAVE_END_NAMESPACE(octave)