]> git.xonotic.org Git - xonotic/netradiant.git/blobdiff - radiant/patchdialog.cpp
reformat code! now the code is only ugly on the *inside*
[xonotic/netradiant.git] / radiant / patchdialog.cpp
index 6a29298d592f8de1a599863a37669c572c66af22..ba5349185878fcfe478fe8494f5a4bb4fc4352d4 100644 (file)
 #include <gdk/gdkkeysyms.h>
 
 // the increment we are using for the patch inspector (this is saved in the prefs)
-struct pi_globals_t
-{
-       float shift[2];
-       float scale[2];
-       float rotate;
-
-       pi_globals_t(){
-               shift[0] = 8.0f;
-               shift[1] = 8.0f;
-               scale[0] = 0.5f;
-               scale[1] = 0.5f;
-               rotate = 45.0f;
-       }
+struct pi_globals_t {
+    float shift[2];
+    float scale[2];
+    float rotate;
+
+    pi_globals_t()
+    {
+        shift[0] = 8.0f;
+        shift[1] = 8.0f;
+        scale[0] = 0.5f;
+        scale[1] = 0.5f;
+        rotate = 45.0f;
+    }
 };
 
 pi_globals_t g_pi_globals;
 
-class PatchFixedSubdivisions
-{
+class PatchFixedSubdivisions {
 public:
-PatchFixedSubdivisions() : m_enabled( false ), m_x( 0 ), m_y( 0 ){
-}
-PatchFixedSubdivisions( bool enabled, std::size_t x, std::size_t y ) : m_enabled( enabled ), m_x( x ), m_y( y ){
-}
-bool m_enabled;
-std::size_t m_x;
-std::size_t m_y;
+    PatchFixedSubdivisions() : m_enabled(false), m_x(0), m_y(0)
+    {
+    }
+
+    PatchFixedSubdivisions(bool enabled, std::size_t x, std::size_t y) : m_enabled(enabled), m_x(x), m_y(y)
+    {
+    }
+
+    bool m_enabled;
+    std::size_t m_x;
+    std::size_t m_y;
 };
 
-void Patch_getFixedSubdivisions( const Patch& patch, PatchFixedSubdivisions& subdivisions ){
-       subdivisions.m_enabled = patch.m_patchDef3;
-       subdivisions.m_x = patch.m_subdivisions_x;
-       subdivisions.m_y = patch.m_subdivisions_y;
+void Patch_getFixedSubdivisions(const Patch &patch, PatchFixedSubdivisions &subdivisions)
+{
+    subdivisions.m_enabled = patch.m_patchDef3;
+    subdivisions.m_x = patch.m_subdivisions_x;
+    subdivisions.m_y = patch.m_subdivisions_y;
 }
 
 const std::size_t MAX_PATCH_SUBDIVISIONS = 32;
 
-void Patch_setFixedSubdivisions( Patch& patch, const PatchFixedSubdivisions& subdivisions ){
-       patch.undoSave();
-
-       patch.m_patchDef3 = subdivisions.m_enabled;
-       patch.m_subdivisions_x = subdivisions.m_x;
-       patch.m_subdivisions_y = subdivisions.m_y;
-
-       if ( patch.m_subdivisions_x == 0 ) {
-               patch.m_subdivisions_x = 4;
-       }
-       else if ( patch.m_subdivisions_x > MAX_PATCH_SUBDIVISIONS ) {
-               patch.m_subdivisions_x = MAX_PATCH_SUBDIVISIONS;
-       }
-       if ( patch.m_subdivisions_y == 0 ) {
-               patch.m_subdivisions_y = 4;
-       }
-       else if ( patch.m_subdivisions_y > MAX_PATCH_SUBDIVISIONS ) {
-               patch.m_subdivisions_y = MAX_PATCH_SUBDIVISIONS;
-       }
-
-       SceneChangeNotify();
-       Patch_textureChanged();
-       patch.controlPointsChanged();
+void Patch_setFixedSubdivisions(Patch &patch, const PatchFixedSubdivisions &subdivisions)
+{
+    patch.undoSave();
+
+    patch.m_patchDef3 = subdivisions.m_enabled;
+    patch.m_subdivisions_x = subdivisions.m_x;
+    patch.m_subdivisions_y = subdivisions.m_y;
+
+    if (patch.m_subdivisions_x == 0) {
+        patch.m_subdivisions_x = 4;
+    } else if (patch.m_subdivisions_x > MAX_PATCH_SUBDIVISIONS) {
+        patch.m_subdivisions_x = MAX_PATCH_SUBDIVISIONS;
+    }
+    if (patch.m_subdivisions_y == 0) {
+        patch.m_subdivisions_y = 4;
+    } else if (patch.m_subdivisions_y > MAX_PATCH_SUBDIVISIONS) {
+        patch.m_subdivisions_y = MAX_PATCH_SUBDIVISIONS;
+    }
+
+    SceneChangeNotify();
+    Patch_textureChanged();
+    patch.controlPointsChanged();
 }
 
-class PatchGetFixedSubdivisions
-{
-PatchFixedSubdivisions& m_subdivisions;
+class PatchGetFixedSubdivisions {
+    PatchFixedSubdivisions &m_subdivisions;
 public:
-PatchGetFixedSubdivisions( PatchFixedSubdivisions& subdivisions ) : m_subdivisions( subdivisions ){
-}
-void operator()( Patch& patch ){
-       Patch_getFixedSubdivisions( patch, m_subdivisions );
-       SceneChangeNotify();
-}
+    PatchGetFixedSubdivisions(PatchFixedSubdivisions &subdivisions) : m_subdivisions(subdivisions)
+    {
+    }
+
+    void operator()(Patch &patch)
+    {
+        Patch_getFixedSubdivisions(patch, m_subdivisions);
+        SceneChangeNotify();
+    }
 };
 
-void Scene_PatchGetFixedSubdivisions( PatchFixedSubdivisions& subdivisions ){
+void Scene_PatchGetFixedSubdivisions(PatchFixedSubdivisions &subdivisions)
+{
 #if 1
-       if ( GlobalSelectionSystem().countSelected() != 0 ) {
-               Patch* patch = Node_getPatch( GlobalSelectionSystem().ultimateSelected().path().top() );
-               if ( patch != 0 ) {
-                       Patch_getFixedSubdivisions( *patch, subdivisions );
-               }
-       }
+    if (GlobalSelectionSystem().countSelected() != 0) {
+        Patch *patch = Node_getPatch(GlobalSelectionSystem().ultimateSelected().path().top());
+        if (patch != 0) {
+            Patch_getFixedSubdivisions(*patch, subdivisions);
+        }
+    }
 #else
-       Scene_forEachVisibleSelectedPatch( PatchGetFixedSubdivisions( subdivisions ) );
+    Scene_forEachVisibleSelectedPatch( PatchGetFixedSubdivisions( subdivisions ) );
 #endif
 }
 
-void Scene_PatchSetFixedSubdivisions( const PatchFixedSubdivisions& subdivisions ){
-       UndoableCommand command( "patchSetFixedSubdivisions" );
-       Scene_forEachVisibleSelectedPatch([&](Patch &patch) {
-               Patch_setFixedSubdivisions(patch, subdivisions);
-       });
+void Scene_PatchSetFixedSubdivisions(const PatchFixedSubdivisions &subdivisions)
+{
+    UndoableCommand command("patchSetFixedSubdivisions");
+    Scene_forEachVisibleSelectedPatch([&](Patch &patch) {
+        Patch_setFixedSubdivisions(patch, subdivisions);
+    });
 }
 
 
-class Subdivisions
-{
+class Subdivisions {
 public:
-ui::CheckButton m_enabled;
-ui::Entry m_horizontal;
-ui::Entry m_vertical;
-Subdivisions() : m_enabled( ui::null ), m_horizontal( ui::null ), m_vertical( ui::null ){
-}
-void update(){
-       PatchFixedSubdivisions subdivisions;
-       Scene_PatchGetFixedSubdivisions( subdivisions );
-
-       toggle_button_set_active_no_signal( m_enabled, subdivisions.m_enabled );
-
-       if ( subdivisions.m_enabled ) {
-               entry_set_int( m_horizontal, static_cast<int>( subdivisions.m_x ) );
-               entry_set_int( m_vertical, static_cast<int>( subdivisions.m_y ) );
-               gtk_widget_set_sensitive( m_horizontal , TRUE );
-               gtk_widget_set_sensitive( m_vertical , TRUE );
-       }
-       else
-       {
-               m_horizontal.text("");
-               m_vertical.text("");
-               gtk_widget_set_sensitive( m_horizontal , FALSE );
-               gtk_widget_set_sensitive( m_vertical , FALSE );
-       }
-}
-void cancel(){
-       update();
-}
-typedef MemberCaller<Subdivisions, void(), &Subdivisions::cancel> CancelCaller;
-void apply(){
-       Scene_PatchSetFixedSubdivisions(
-               PatchFixedSubdivisions(
-                       gtk_toggle_button_get_active( GTK_TOGGLE_BUTTON( m_enabled ) ) != FALSE,
-                       static_cast<std::size_t>( entry_get_int( m_horizontal ) ),
-                       static_cast<std::size_t>( entry_get_int( m_vertical ) )
-                       )
-               );
-}
-typedef MemberCaller<Subdivisions, void(), &Subdivisions::apply> ApplyCaller;
-static void applyGtk( ui::ToggleButton toggle, Subdivisions* self ){
-       self->apply();
-}
+    ui::CheckButton m_enabled;
+    ui::Entry m_horizontal;
+    ui::Entry m_vertical;
+
+    Subdivisions() : m_enabled(ui::null), m_horizontal(ui::null), m_vertical(ui::null)
+    {
+    }
+
+    void update()
+    {
+        PatchFixedSubdivisions subdivisions;
+        Scene_PatchGetFixedSubdivisions(subdivisions);
+
+        toggle_button_set_active_no_signal(m_enabled, subdivisions.m_enabled);
+
+        if (subdivisions.m_enabled) {
+            entry_set_int(m_horizontal, static_cast<int>( subdivisions.m_x ));
+            entry_set_int(m_vertical, static_cast<int>( subdivisions.m_y ));
+            gtk_widget_set_sensitive(m_horizontal, TRUE);
+            gtk_widget_set_sensitive(m_vertical, TRUE);
+        } else {
+            m_horizontal.text("");
+            m_vertical.text("");
+            gtk_widget_set_sensitive(m_horizontal, FALSE);
+            gtk_widget_set_sensitive(m_vertical, FALSE);
+        }
+    }
+
+    void cancel()
+    {
+        update();
+    }
+
+    typedef MemberCaller<Subdivisions, void(), &Subdivisions::cancel> CancelCaller;
+
+    void apply()
+    {
+        Scene_PatchSetFixedSubdivisions(
+                PatchFixedSubdivisions(
+                        gtk_toggle_button_get_active(GTK_TOGGLE_BUTTON(m_enabled)) != FALSE,
+                        static_cast<std::size_t>( entry_get_int(m_horizontal)),
+                        static_cast<std::size_t>( entry_get_int(m_vertical))
+                )
+        );
+    }
+
+    typedef MemberCaller<Subdivisions, void(), &Subdivisions::apply> ApplyCaller;
+
+    static void applyGtk(ui::ToggleButton toggle, Subdivisions *self)
+    {
+        self->apply();
+    }
 };
 
-class PatchInspector : public Dialog
-{
-ui::Window BuildDialog();
-Subdivisions m_subdivisions;
-NonModalEntry m_horizontalSubdivisionsEntry;
-NonModalEntry m_verticalSubdivisionsEntry;
+class PatchInspector : public Dialog {
+    ui::Window BuildDialog();
+
+    Subdivisions m_subdivisions;
+    NonModalEntry m_horizontalSubdivisionsEntry;
+    NonModalEntry m_verticalSubdivisionsEntry;
 public:
-IdleDraw m_idleDraw;
-WindowPositionTracker m_position_tracker;
+    IdleDraw m_idleDraw;
+    WindowPositionTracker m_position_tracker;
 
-Patch *m_Patch;
+    Patch *m_Patch;
 
-CopiedString m_strName;
-float m_fS;
-float m_fT;
-float m_fX;
-float m_fY;
-float m_fZ;
+    CopiedString m_strName;
+    float m_fS;
+    float m_fT;
+    float m_fX;
+    float m_fY;
+    float m_fZ;
 /*  float      m_fHScale;
    float       m_fHShift;
    float       m_fRotate;
    float       m_fVScale;
    float       m_fVShift; */
-int m_nCol;
-int m_nRow;
-ui::ComboBoxText m_pRowCombo{ui::null};
-ui::ComboBoxText m_pColCombo{ui::null};
-std::size_t m_countRows;
-std::size_t m_countCols;
+    int m_nCol;
+    int m_nRow;
+    ui::ComboBoxText m_pRowCombo{ui::null};
+    ui::ComboBoxText m_pColCombo{ui::null};
+    std::size_t m_countRows;
+    std::size_t m_countCols;
 
 // turn on/off processing of the "changed" "value_changed" messages
 // (need to turn off when we are feeding data in)
 // NOTE: much more simple than blocking signals
-bool m_bListenChanged;
-
-PatchInspector() :
-       m_horizontalSubdivisionsEntry( Subdivisions::ApplyCaller( m_subdivisions ), Subdivisions::CancelCaller( m_subdivisions ) ),
-       m_verticalSubdivisionsEntry( Subdivisions::ApplyCaller( m_subdivisions ), Subdivisions::CancelCaller( m_subdivisions ) ),
-       m_idleDraw( MemberCaller<PatchInspector, void(), &PatchInspector::GetPatchInfo>( *this ) ){
-       m_fS = 0.0f;
-       m_fT = 0.0f;
-       m_fX = 0.0f;
-       m_fY = 0.0f;
-       m_fZ = 0.0f;
-       m_nCol = 0;
-       m_nRow = 0;
-       m_countRows = 0;
-       m_countCols = 0;
-       m_Patch = 0;
-       m_bListenChanged = true;
-
-       m_position_tracker.setPosition( c_default_window_pos );
-}
-
-bool visible(){
-       return GetWidget().visible();
-}
+    bool m_bListenChanged;
+
+    PatchInspector() :
+            m_horizontalSubdivisionsEntry(Subdivisions::ApplyCaller(m_subdivisions),
+                                          Subdivisions::CancelCaller(m_subdivisions)),
+            m_verticalSubdivisionsEntry(Subdivisions::ApplyCaller(m_subdivisions),
+                                        Subdivisions::CancelCaller(m_subdivisions)),
+            m_idleDraw(MemberCaller<PatchInspector, void(), &PatchInspector::GetPatchInfo>(*this))
+    {
+        m_fS = 0.0f;
+        m_fT = 0.0f;
+        m_fX = 0.0f;
+        m_fY = 0.0f;
+        m_fZ = 0.0f;
+        m_nCol = 0;
+        m_nRow = 0;
+        m_countRows = 0;
+        m_countCols = 0;
+        m_Patch = 0;
+        m_bListenChanged = true;
+
+        m_position_tracker.setPosition(c_default_window_pos);
+    }
+
+    bool visible()
+    {
+        return GetWidget().visible();
+    }
 
 //  void UpdateInfo();
 //  void SetPatchInfo();
-void GetPatchInfo();
-void UpdateSpinners( bool bUp, int nID );
+    void GetPatchInfo();
+
+    void UpdateSpinners(bool bUp, int nID);
+
 // read the current patch on map and initialize m_fX m_fY accordingly
-void UpdateRowColInfo();
+    void UpdateRowColInfo();
+
 // sync the dialog our internal data structures
 // depending on the flag it will read or write
 // we use m_nCol m_nRow m_fX m_fY m_fZ m_fS m_fT m_strName
 // (NOTE: this doesn't actually commit stuff to the map or read from it)
-void importData();
-void exportData();
+    void importData();
+
+    void exportData();
 };
 
 PatchInspector g_PatchInspector;
 
-void PatchInspector_constructWindow( ui::Window main_window ){
-       g_PatchInspector.m_parent = main_window;
-       g_PatchInspector.Create();
+void PatchInspector_constructWindow(ui::Window main_window)
+{
+    g_PatchInspector.m_parent = main_window;
+    g_PatchInspector.Create();
 }
-void PatchInspector_destroyWindow(){
-       g_PatchInspector.Destroy();
+
+void PatchInspector_destroyWindow()
+{
+    g_PatchInspector.Destroy();
 }
 
-void PatchInspector_queueDraw(){
-       if ( g_PatchInspector.visible() ) {
-               g_PatchInspector.m_idleDraw.queueDraw();
-       }
+void PatchInspector_queueDraw()
+{
+    if (g_PatchInspector.visible()) {
+        g_PatchInspector.m_idleDraw.queueDraw();
+    }
 }
 
-void DoPatchInspector(){
-       g_PatchInspector.GetPatchInfo();
-       if ( !g_PatchInspector.visible() ) {
-               g_PatchInspector.ShowDlg();
-       }
+void DoPatchInspector()
+{
+    g_PatchInspector.GetPatchInfo();
+    if (!g_PatchInspector.visible()) {
+        g_PatchInspector.ShowDlg();
+    }
 }
 
-void PatchInspector_toggleShown(){
-       if ( g_PatchInspector.visible() ) {
-               g_PatchInspector.m_Patch = 0;
-               g_PatchInspector.HideDlg();
-       }
-       else{
-               DoPatchInspector();
-       }
+void PatchInspector_toggleShown()
+{
+    if (g_PatchInspector.visible()) {
+        g_PatchInspector.m_Patch = 0;
+        g_PatchInspector.HideDlg();
+    } else {
+        DoPatchInspector();
+    }
 }
 
 
@@ -302,750 +331,769 @@ void PatchInspector_toggleShown(){
 // static functions
 
 // memorize the current state (that is don't try to undo our do before changing something else)
-static void OnApply( ui::Widget widget, gpointer data ){
-       g_PatchInspector.exportData();
-       if ( g_PatchInspector.m_Patch != 0 ) {
-               UndoableCommand command( "patchSetTexture" );
-               g_PatchInspector.m_Patch->undoSave();
-
-               if ( !texdef_name_valid( g_PatchInspector.m_strName.c_str() ) ) {
-                       globalErrorStream() << "invalid texture name '" << g_PatchInspector.m_strName.c_str() << "'\n";
-                       g_PatchInspector.m_strName = texdef_name_default();
-               }
-               g_PatchInspector.m_Patch->SetShader( g_PatchInspector.m_strName.c_str() );
-
-               std::size_t r = g_PatchInspector.m_nRow;
-               std::size_t c = g_PatchInspector.m_nCol;
-               if ( r < g_PatchInspector.m_Patch->getHeight()
-                        && c < g_PatchInspector.m_Patch->getWidth() ) {
-                       PatchControl& p = g_PatchInspector.m_Patch->ctrlAt( r,c );
-                       p.m_vertex[0] = g_PatchInspector.m_fX;
-                       p.m_vertex[1] = g_PatchInspector.m_fY;
-                       p.m_vertex[2] = g_PatchInspector.m_fZ;
-                       p.m_texcoord[0] = g_PatchInspector.m_fS;
-                       p.m_texcoord[1] = g_PatchInspector.m_fT;
-                       g_PatchInspector.m_Patch->controlPointsChanged();
-               }
-       }
+static void OnApply(ui::Widget widget, gpointer data)
+{
+    g_PatchInspector.exportData();
+    if (g_PatchInspector.m_Patch != 0) {
+        UndoableCommand command("patchSetTexture");
+        g_PatchInspector.m_Patch->undoSave();
+
+        if (!texdef_name_valid(g_PatchInspector.m_strName.c_str())) {
+            globalErrorStream() << "invalid texture name '" << g_PatchInspector.m_strName.c_str() << "'\n";
+            g_PatchInspector.m_strName = texdef_name_default();
+        }
+        g_PatchInspector.m_Patch->SetShader(g_PatchInspector.m_strName.c_str());
+
+        std::size_t r = g_PatchInspector.m_nRow;
+        std::size_t c = g_PatchInspector.m_nCol;
+        if (r < g_PatchInspector.m_Patch->getHeight()
+            && c < g_PatchInspector.m_Patch->getWidth()) {
+            PatchControl &p = g_PatchInspector.m_Patch->ctrlAt(r, c);
+            p.m_vertex[0] = g_PatchInspector.m_fX;
+            p.m_vertex[1] = g_PatchInspector.m_fY;
+            p.m_vertex[2] = g_PatchInspector.m_fZ;
+            p.m_texcoord[0] = g_PatchInspector.m_fS;
+            p.m_texcoord[1] = g_PatchInspector.m_fT;
+            g_PatchInspector.m_Patch->controlPointsChanged();
+        }
+    }
 }
 
-static void OnSelchangeComboColRow( ui::Widget widget, gpointer data ){
-       if ( !g_PatchInspector.m_bListenChanged ) {
-               return;
-       }
-       // retrieve the current m_nRow and m_nCol, other params are not relevant
-       g_PatchInspector.exportData();
-       // read the changed values ourselves
-       g_PatchInspector.UpdateRowColInfo();
-       // now reflect our changes
-       g_PatchInspector.importData();
+static void OnSelchangeComboColRow(ui::Widget widget, gpointer data)
+{
+    if (!g_PatchInspector.m_bListenChanged) {
+        return;
+    }
+    // retrieve the current m_nRow and m_nCol, other params are not relevant
+    g_PatchInspector.exportData();
+    // read the changed values ourselves
+    g_PatchInspector.UpdateRowColInfo();
+    // now reflect our changes
+    g_PatchInspector.importData();
 }
 
-void Scene_PatchTileTexture_Selected( scene::Graph& graph, float s, float t ){
-       Scene_forEachVisibleSelectedPatch([&](Patch &patch) {
-               patch.SetTextureRepeat(s, t);
-       });
-       SceneChangeNotify();
+void Scene_PatchTileTexture_Selected(scene::Graph &graph, float s, float t)
+{
+    Scene_forEachVisibleSelectedPatch([&](Patch &patch) {
+        patch.SetTextureRepeat(s, t);
+    });
+    SceneChangeNotify();
 }
 
-static void OnBtnPatchdetails( ui::Widget widget, gpointer data ){
-       Patch_CapTexture();
+static void OnBtnPatchdetails(ui::Widget widget, gpointer data)
+{
+    Patch_CapTexture();
 }
 
-static void OnBtnPatchfit( ui::Widget widget, gpointer data ){
-       Patch_FitTexture();
+static void OnBtnPatchfit(ui::Widget widget, gpointer data)
+{
+    Patch_FitTexture();
 }
 
-static void OnBtnPatchnatural( ui::Widget widget, gpointer data ){
-       Patch_NaturalTexture();
+static void OnBtnPatchnatural(ui::Widget widget, gpointer data)
+{
+    Patch_NaturalTexture();
 }
 
-static void OnBtnPatchreset( ui::Widget widget, gpointer data ){
-       Patch_ResetTexture();
+static void OnBtnPatchreset(ui::Widget widget, gpointer data)
+{
+    Patch_ResetTexture();
 }
 
-static void OnBtnPatchFlipX( ui::Widget widget, gpointer data ){
-       Patch_FlipTextureX();
+static void OnBtnPatchFlipX(ui::Widget widget, gpointer data)
+{
+    Patch_FlipTextureX();
 }
 
-static void OnBtnPatchFlipY( ui::Widget widget, gpointer data ){
-       Patch_FlipTextureY();
+static void OnBtnPatchFlipY(ui::Widget widget, gpointer data)
+{
+    Patch_FlipTextureY();
 }
 
-void Scene_PatchRotateTexture_Selected( scene::Graph& graph, float angle ){
-       Scene_forEachVisibleSelectedPatch([&](Patch &patch) {
-               patch.RotateTexture(angle);
-       });
+void Scene_PatchRotateTexture_Selected(scene::Graph &graph, float angle)
+{
+    Scene_forEachVisibleSelectedPatch([&](Patch &patch) {
+        patch.RotateTexture(angle);
+    });
 }
 
-float Patch_convertScale( float scale ){
-       if ( scale > 0 ) {
-               return scale;
-       }
-       if ( scale < 0 ) {
-               return -1 / scale;
-       }
-       return 1;
+float Patch_convertScale(float scale)
+{
+    if (scale > 0) {
+        return scale;
+    }
+    if (scale < 0) {
+        return -1 / scale;
+    }
+    return 1;
 }
 
-void Scene_PatchScaleTexture_Selected( scene::Graph& graph, float s, float t ){
-       s = Patch_convertScale(s);
-       t = Patch_convertScale(t);
-       Scene_forEachVisibleSelectedPatch([&](Patch &patch) {
-               patch.ScaleTexture(s, t);
-       });
+void Scene_PatchScaleTexture_Selected(scene::Graph &graph, float s, float t)
+{
+    s = Patch_convertScale(s);
+    t = Patch_convertScale(t);
+    Scene_forEachVisibleSelectedPatch([&](Patch &patch) {
+        patch.ScaleTexture(s, t);
+    });
 }
 
-void Scene_PatchTranslateTexture_Selected( scene::Graph& graph, float s, float t ){
-       Scene_forEachVisibleSelectedPatch([&](Patch &patch) {
-               patch.TranslateTexture(s, t);
-       });
+void Scene_PatchTranslateTexture_Selected(scene::Graph &graph, float s, float t)
+{
+    Scene_forEachVisibleSelectedPatch([&](Patch &patch) {
+        patch.TranslateTexture(s, t);
+    });
 }
 
-static void OnBtnPatchAutoCap( ui::Widget widget, gpointer data ){
-       Patch_AutoCapTexture();
+static void OnBtnPatchAutoCap(ui::Widget widget, gpointer data)
+{
+    Patch_AutoCapTexture();
 }
 
-static void OnSpinChanged(ui::Adjustment adj, gpointer data ){
-       texdef_t td;
-
-       td.rotate = 0;
-       td.scale[0] = td.scale[1] = 0;
-       td.shift[0] = td.shift[1] = 0;
-
-       if ( gtk_adjustment_get_value(adj) == 0 ) {
-               return;
-       }
-
-       if ( adj == g_object_get_data( G_OBJECT( g_PatchInspector.GetWidget() ), "hshift_adj" ) ) {
-               g_pi_globals.shift[0] = static_cast<float>( atof( gtk_entry_get_text( GTK_ENTRY( data ) ) ) );
-
-               if ( gtk_adjustment_get_value(adj) > 0 ) {
-                       td.shift[0] = g_pi_globals.shift[0];
-               }
-               else{
-                       td.shift[0] = -g_pi_globals.shift[0];
-               }
-       }
-       else if ( adj == g_object_get_data( G_OBJECT( g_PatchInspector.GetWidget() ), "vshift_adj" ) ) {
-               g_pi_globals.shift[1] = static_cast<float>( atof( gtk_entry_get_text( GTK_ENTRY( data ) ) ) );
-
-               if ( gtk_adjustment_get_value(adj) > 0 ) {
-                       td.shift[1] = g_pi_globals.shift[1];
-               }
-               else{
-                       td.shift[1] = -g_pi_globals.shift[1];
-               }
-       }
-       else if ( adj == g_object_get_data( G_OBJECT( g_PatchInspector.GetWidget() ), "hscale_adj" ) ) {
-               g_pi_globals.scale[0] = static_cast<float>( atof( gtk_entry_get_text( GTK_ENTRY( data ) ) ) );
-               if ( g_pi_globals.scale[0] == 0.0f ) {
-                       return;
-               }
-               if ( gtk_adjustment_get_value(adj) > 0 ) {
-                       td.scale[0] = g_pi_globals.scale[0];
-               }
-               else{
-                       td.scale[0] = -g_pi_globals.scale[0];
-               }
-       }
-       else if ( adj == g_object_get_data( G_OBJECT( g_PatchInspector.GetWidget() ), "vscale_adj" ) ) {
-               g_pi_globals.scale[1] = static_cast<float>( atof( gtk_entry_get_text( GTK_ENTRY( data ) ) ) );
-               if ( g_pi_globals.scale[1] == 0.0f ) {
-                       return;
-               }
-               if ( gtk_adjustment_get_value(adj) > 0 ) {
-                       td.scale[1] = g_pi_globals.scale[1];
-               }
-               else{
-                       td.scale[1] = -g_pi_globals.scale[1];
-               }
-       }
-       else if ( adj == g_object_get_data( G_OBJECT( g_PatchInspector.GetWidget() ), "rotate_adj" ) ) {
-               g_pi_globals.rotate = static_cast<float>( atof( gtk_entry_get_text( GTK_ENTRY( data ) ) ) );
-
-               if ( gtk_adjustment_get_value(adj) > 0 ) {
-                       td.rotate = g_pi_globals.rotate;
-               }
-               else{
-                       td.rotate = -g_pi_globals.rotate;
-               }
-       }
-
-       gtk_adjustment_set_value(adj, 0);
-
-       // will scale shift rotate the patch accordingly
-
-
-       if ( td.shift[0] || td.shift[1] ) {
-               UndoableCommand command( "patchTranslateTexture" );
-               Scene_PatchTranslateTexture_Selected( GlobalSceneGraph(), td.shift[0], td.shift[1] );
-       }
-       else if ( td.scale[0] || td.scale[1] ) {
-               UndoableCommand command( "patchScaleTexture" );
-               Scene_PatchScaleTexture_Selected( GlobalSceneGraph(), td.scale[0], td.scale[1] );
-       }
-       else if ( td.rotate ) {
-               UndoableCommand command( "patchRotateTexture" );
-               Scene_PatchRotateTexture_Selected( GlobalSceneGraph(), td.rotate );
-       }
-
-       // update the point-by-point view
-       OnSelchangeComboColRow( ui::root, 0 );
+static void OnSpinChanged(ui::Adjustment adj, gpointer data)
+{
+    texdef_t td;
+
+    td.rotate = 0;
+    td.scale[0] = td.scale[1] = 0;
+    td.shift[0] = td.shift[1] = 0;
+
+    if (gtk_adjustment_get_value(adj) == 0) {
+        return;
+    }
+
+    if (adj == g_object_get_data(G_OBJECT(g_PatchInspector.GetWidget()), "hshift_adj")) {
+        g_pi_globals.shift[0] = static_cast<float>( atof(gtk_entry_get_text(GTK_ENTRY(data))));
+
+        if (gtk_adjustment_get_value(adj) > 0) {
+            td.shift[0] = g_pi_globals.shift[0];
+        } else {
+            td.shift[0] = -g_pi_globals.shift[0];
+        }
+    } else if (adj == g_object_get_data(G_OBJECT(g_PatchInspector.GetWidget()), "vshift_adj")) {
+        g_pi_globals.shift[1] = static_cast<float>( atof(gtk_entry_get_text(GTK_ENTRY(data))));
+
+        if (gtk_adjustment_get_value(adj) > 0) {
+            td.shift[1] = g_pi_globals.shift[1];
+        } else {
+            td.shift[1] = -g_pi_globals.shift[1];
+        }
+    } else if (adj == g_object_get_data(G_OBJECT(g_PatchInspector.GetWidget()), "hscale_adj")) {
+        g_pi_globals.scale[0] = static_cast<float>( atof(gtk_entry_get_text(GTK_ENTRY(data))));
+        if (g_pi_globals.scale[0] == 0.0f) {
+            return;
+        }
+        if (gtk_adjustment_get_value(adj) > 0) {
+            td.scale[0] = g_pi_globals.scale[0];
+        } else {
+            td.scale[0] = -g_pi_globals.scale[0];
+        }
+    } else if (adj == g_object_get_data(G_OBJECT(g_PatchInspector.GetWidget()), "vscale_adj")) {
+        g_pi_globals.scale[1] = static_cast<float>( atof(gtk_entry_get_text(GTK_ENTRY(data))));
+        if (g_pi_globals.scale[1] == 0.0f) {
+            return;
+        }
+        if (gtk_adjustment_get_value(adj) > 0) {
+            td.scale[1] = g_pi_globals.scale[1];
+        } else {
+            td.scale[1] = -g_pi_globals.scale[1];
+        }
+    } else if (adj == g_object_get_data(G_OBJECT(g_PatchInspector.GetWidget()), "rotate_adj")) {
+        g_pi_globals.rotate = static_cast<float>( atof(gtk_entry_get_text(GTK_ENTRY(data))));
+
+        if (gtk_adjustment_get_value(adj) > 0) {
+            td.rotate = g_pi_globals.rotate;
+        } else {
+            td.rotate = -g_pi_globals.rotate;
+        }
+    }
+
+    gtk_adjustment_set_value(adj, 0);
+
+    // will scale shift rotate the patch accordingly
+
+
+    if (td.shift[0] || td.shift[1]) {
+        UndoableCommand command("patchTranslateTexture");
+        Scene_PatchTranslateTexture_Selected(GlobalSceneGraph(), td.shift[0], td.shift[1]);
+    } else if (td.scale[0] || td.scale[1]) {
+        UndoableCommand command("patchScaleTexture");
+        Scene_PatchScaleTexture_Selected(GlobalSceneGraph(), td.scale[0], td.scale[1]);
+    } else if (td.rotate) {
+        UndoableCommand command("patchRotateTexture");
+        Scene_PatchRotateTexture_Selected(GlobalSceneGraph(), td.rotate);
+    }
+
+    // update the point-by-point view
+    OnSelchangeComboColRow(ui::root, 0);
 }
 
-static gint OnDialogKey( ui::Widget widget, GdkEventKey* event, gpointer data ){
-       if ( event->keyval == GDK_KEY_Return ) {
-               OnApply( ui::root, 0 );
-               return TRUE;
-       }
-       else if ( event->keyval == GDK_KEY_Escape ) {
-               g_PatchInspector.GetPatchInfo();
-               return TRUE;
-       }
-       return FALSE;
+static gint OnDialogKey(ui::Widget widget, GdkEventKey *event, gpointer data)
+{
+    if (event->keyval == GDK_KEY_Return) {
+        OnApply(ui::root, 0);
+        return TRUE;
+    } else if (event->keyval == GDK_KEY_Escape) {
+        g_PatchInspector.GetPatchInfo();
+        return TRUE;
+    }
+    return FALSE;
 }
 
 // =============================================================================
 // PatchInspector class
 
-ui::Window PatchInspector::BuildDialog(){
-       ui::Window window = ui::Window(create_floating_window( "Patch Properties", m_parent ));
-
-       m_position_tracker.connect( window );
-
-       global_accel_connect_window( window );
-
-       window_connect_focus_in_clear_focus_widget( window );
-
-
-       {
-               auto vbox = ui::VBox( FALSE, 5 );
-               gtk_container_set_border_width( GTK_CONTAINER( vbox ), 5 );
-               vbox.show();
-               window.add(vbox);
-               {
-                       auto hbox = ui::HBox( FALSE, 5 );
-                       hbox.show();
-                       vbox.pack_start( hbox, TRUE, TRUE, 0 );
-                       {
-                               auto vbox2 = ui::VBox( FALSE, 0 );
-                               gtk_container_set_border_width( GTK_CONTAINER( vbox2 ), 0 );
-                               vbox2.show();
-                               hbox.pack_start( vbox2, TRUE, TRUE, 0 );
-                               {
-                                       auto frame = ui::Frame( "Details" );
-                                       frame.show();
-                                       vbox2.pack_start( frame, TRUE, TRUE, 0 );
-                                       {
-                                               auto vbox3 = ui::VBox( FALSE, 5 );
-                                               gtk_container_set_border_width( GTK_CONTAINER( vbox3 ), 5 );
-                                               vbox3.show();
-                                               frame.add(vbox3);
-                                               {
-                                                       auto table = ui::Table( 2, 2, FALSE );
-                                                       table.show();
-                                                       vbox3.pack_start( table, TRUE, TRUE, 0 );
-                                                       gtk_table_set_row_spacings( table, 5 );
-                                                       gtk_table_set_col_spacings( table, 5 );
-                                                       {
-                                                               auto label = ui::Label( "Row:" );
-                                                               label.show();
-                                table.attach(label, {0, 1, 0, 1}, {(GtkAttachOptions) (GTK_EXPAND | GTK_FILL), (GtkAttachOptions) (0)}, {0, 0});
-                                                       }
-                                                       {
-                                                               auto label = ui::Label( "Column:" );
-                                                               label.show();
-                                table.attach(label, {1, 2, 0, 1}, {(GtkAttachOptions) (GTK_EXPAND | GTK_FILL), (GtkAttachOptions) (0)}, {0, 0});
-                                                       }
-                                                       {
-                                                               auto combo = ui::ComboBoxText(ui::New);
-                                                               combo.connect( "changed", G_CALLBACK( OnSelchangeComboColRow ), this );
-                                                               AddDialogData( combo, m_nRow );
-
-                                                               combo.show();
-                                table.attach(combo, {0, 1, 1, 2}, {(GtkAttachOptions) (GTK_EXPAND | GTK_FILL), (GtkAttachOptions) (0)}, {0, 0});
-                                                               combo.dimensions(60, -1);
-                                                               m_pRowCombo = combo;
-                                                       }
-
-                                                       {
-                                                               auto combo = ui::ComboBoxText(ui::New);
-                                                               combo.connect( "changed", G_CALLBACK( OnSelchangeComboColRow ), this );
-                                                               AddDialogData( combo, m_nCol );
-
-                                                               combo.show();
-                                table.attach(combo, {1, 2, 1, 2}, {(GtkAttachOptions) (GTK_EXPAND | GTK_FILL), (GtkAttachOptions) (0)}, {0, 0});
-                                                               combo.dimensions(60, -1);
-                                                               m_pColCombo = combo;
-                                                       }
-                                               }
-                                               auto table = ui::Table( 5, 2, FALSE );
-                                               table.show();
-                                               vbox3.pack_start( table, TRUE, TRUE, 0 );
-                                               gtk_table_set_row_spacings( table, 5 );
-                                               gtk_table_set_col_spacings( table, 5 );
-                                               {
-                                                       auto label = ui::Label( "X:" );
-                                                       label.show();
+ui::Window PatchInspector::BuildDialog()
+{
+    ui::Window window = ui::Window(create_floating_window("Patch Properties", m_parent));
+
+    m_position_tracker.connect(window);
+
+    global_accel_connect_window(window);
+
+    window_connect_focus_in_clear_focus_widget(window);
+
+
+    {
+        auto vbox = ui::VBox(FALSE, 5);
+        gtk_container_set_border_width(GTK_CONTAINER(vbox), 5);
+        vbox.show();
+        window.add(vbox);
+        {
+            auto hbox = ui::HBox(FALSE, 5);
+            hbox.show();
+            vbox.pack_start(hbox, TRUE, TRUE, 0);
+            {
+                auto vbox2 = ui::VBox(FALSE, 0);
+                gtk_container_set_border_width(GTK_CONTAINER(vbox2), 0);
+                vbox2.show();
+                hbox.pack_start(vbox2, TRUE, TRUE, 0);
+                {
+                    auto frame = ui::Frame("Details");
+                    frame.show();
+                    vbox2.pack_start(frame, TRUE, TRUE, 0);
+                    {
+                        auto vbox3 = ui::VBox(FALSE, 5);
+                        gtk_container_set_border_width(GTK_CONTAINER(vbox3), 5);
+                        vbox3.show();
+                        frame.add(vbox3);
+                        {
+                            auto table = ui::Table(2, 2, FALSE);
+                            table.show();
+                            vbox3.pack_start(table, TRUE, TRUE, 0);
+                            gtk_table_set_row_spacings(table, 5);
+                            gtk_table_set_col_spacings(table, 5);
+                            {
+                                auto label = ui::Label("Row:");
+                                label.show();
+                                table.attach(label, {0, 1, 0, 1},
+                                             {(GtkAttachOptions) (GTK_EXPAND | GTK_FILL), (GtkAttachOptions) (0)},
+                                             {0, 0});
+                            }
+                            {
+                                auto label = ui::Label("Column:");
+                                label.show();
+                                table.attach(label, {1, 2, 0, 1},
+                                             {(GtkAttachOptions) (GTK_EXPAND | GTK_FILL), (GtkAttachOptions) (0)},
+                                             {0, 0});
+                            }
+                            {
+                                auto combo = ui::ComboBoxText(ui::New);
+                                combo.connect("changed", G_CALLBACK(OnSelchangeComboColRow), this);
+                                AddDialogData(combo, m_nRow);
+
+                                combo.show();
+                                table.attach(combo, {0, 1, 1, 2},
+                                             {(GtkAttachOptions) (GTK_EXPAND | GTK_FILL), (GtkAttachOptions) (0)},
+                                             {0, 0});
+                                combo.dimensions(60, -1);
+                                m_pRowCombo = combo;
+                            }
+
+                            {
+                                auto combo = ui::ComboBoxText(ui::New);
+                                combo.connect("changed", G_CALLBACK(OnSelchangeComboColRow), this);
+                                AddDialogData(combo, m_nCol);
+
+                                combo.show();
+                                table.attach(combo, {1, 2, 1, 2},
+                                             {(GtkAttachOptions) (GTK_EXPAND | GTK_FILL), (GtkAttachOptions) (0)},
+                                             {0, 0});
+                                combo.dimensions(60, -1);
+                                m_pColCombo = combo;
+                            }
+                        }
+                        auto table = ui::Table(5, 2, FALSE);
+                        table.show();
+                        vbox3.pack_start(table, TRUE, TRUE, 0);
+                        gtk_table_set_row_spacings(table, 5);
+                        gtk_table_set_col_spacings(table, 5);
+                        {
+                            auto label = ui::Label("X:");
+                            label.show();
                             table.attach(label, {0, 1, 0, 1}, {GTK_EXPAND | GTK_FILL, 0});
-                                               }
-                                               {
-                                                       auto label = ui::Label( "Y:" );
-                                                       label.show();
+                        }
+                        {
+                            auto label = ui::Label("Y:");
+                            label.show();
                             table.attach(label, {0, 1, 1, 2}, {GTK_EXPAND | GTK_FILL, 0});
-                                               }
-                                               {
-                                                       auto label = ui::Label( "Z:" );
-                                                       label.show();
+                        }
+                        {
+                            auto label = ui::Label("Z:");
+                            label.show();
                             table.attach(label, {0, 1, 2, 3}, {GTK_EXPAND | GTK_FILL, 0});
-                                               }
-                                               {
-                                                       auto label = ui::Label( "S:" );
-                                                       label.show();
+                        }
+                        {
+                            auto label = ui::Label("S:");
+                            label.show();
                             table.attach(label, {0, 1, 3, 4}, {GTK_EXPAND | GTK_FILL, 0});
-                                               }
-                                               {
-                                                       auto label = ui::Label( "T:" );
-                                                       label.show();
+                        }
+                        {
+                            auto label = ui::Label("T:");
+                            label.show();
                             table.attach(label, {0, 1, 4, 5}, {GTK_EXPAND | GTK_FILL, 0});
-                                               }
-                                               {
-                                                       auto entry = ui::Entry(ui::New);
-                                                       entry.show();
+                        }
+                        {
+                            auto entry = ui::Entry(ui::New);
+                            entry.show();
                             table.attach(entry, {1, 2, 0, 1}, {GTK_EXPAND | GTK_FILL, 0});
-                                                       AddDialogData( entry, m_fX );
+                            AddDialogData(entry, m_fX);
 
-                                                       entry.connect( "key_press_event", G_CALLBACK( OnDialogKey ), 0 );
-                                               }
-                                               {
-                                                       auto entry = ui::Entry(ui::New);
-                                                       entry.show();
+                            entry.connect("key_press_event", G_CALLBACK(OnDialogKey), 0);
+                        }
+                        {
+                            auto entry = ui::Entry(ui::New);
+                            entry.show();
                             table.attach(entry, {1, 2, 1, 2}, {GTK_EXPAND | GTK_FILL, 0});
-                                                       AddDialogData( entry, m_fY );
+                            AddDialogData(entry, m_fY);
 
-                                                       entry.connect( "key_press_event", G_CALLBACK( OnDialogKey ), 0 );
-                                               }
-                                               {
-                                                       auto entry = ui::Entry(ui::New);
-                                                       entry.show();
+                            entry.connect("key_press_event", G_CALLBACK(OnDialogKey), 0);
+                        }
+                        {
+                            auto entry = ui::Entry(ui::New);
+                            entry.show();
                             table.attach(entry, {1, 2, 2, 3}, {GTK_EXPAND | GTK_FILL, 0});
-                                                       AddDialogData( entry, m_fZ );
+                            AddDialogData(entry, m_fZ);
 
-                                                       entry.connect( "key_press_event", G_CALLBACK( OnDialogKey ), 0 );
-                                               }
-                                               {
-                                                       auto entry = ui::Entry(ui::New);
-                                                       entry.show();
+                            entry.connect("key_press_event", G_CALLBACK(OnDialogKey), 0);
+                        }
+                        {
+                            auto entry = ui::Entry(ui::New);
+                            entry.show();
                             table.attach(entry, {1, 2, 3, 4}, {GTK_EXPAND | GTK_FILL, 0});
-                                                       AddDialogData( entry, m_fS );
+                            AddDialogData(entry, m_fS);
 
-                                                       entry.connect( "key_press_event", G_CALLBACK( OnDialogKey ), 0 );
-                                               }
-                                               {
-                                                       auto entry = ui::Entry(ui::New);
-                                                       entry.show();
+                            entry.connect("key_press_event", G_CALLBACK(OnDialogKey), 0);
+                        }
+                        {
+                            auto entry = ui::Entry(ui::New);
+                            entry.show();
                             table.attach(entry, {1, 2, 4, 5}, {GTK_EXPAND | GTK_FILL, 0});
-                                                       AddDialogData( entry, m_fT );
-
-                                                       entry.connect( "key_press_event", G_CALLBACK( OnDialogKey ), 0 );
-                                               }
-                                       }
-                               }
-                               if ( g_pGameDescription->mGameType == "doom3" ) {
-                                       auto frame = ui::Frame( "Tesselation" );
-                                       frame.show();
-                                       vbox2.pack_start( frame, TRUE, TRUE, 0 );
-                                       {
-                                               auto vbox3 = ui::VBox( FALSE, 5 );
-                                               gtk_container_set_border_width( GTK_CONTAINER( vbox3 ), 5 );
-                                               vbox3.show();
-                                               frame.add(vbox3);
-                                               {
-                                                       auto table = ui::Table( 3, 2, FALSE );
-                                                       table.show();
-                                                       vbox3.pack_start( table, TRUE, TRUE, 0 );
-                                                       gtk_table_set_row_spacings( table, 5 );
-                                                       gtk_table_set_col_spacings( table, 5 );
-                                                       {
-                                                               auto label = ui::Label( "Fixed" );
-                                                               label.show();
+                            AddDialogData(entry, m_fT);
+
+                            entry.connect("key_press_event", G_CALLBACK(OnDialogKey), 0);
+                        }
+                    }
+                }
+                if (g_pGameDescription->mGameType == "doom3") {
+                    auto frame = ui::Frame("Tesselation");
+                    frame.show();
+                    vbox2.pack_start(frame, TRUE, TRUE, 0);
+                    {
+                        auto vbox3 = ui::VBox(FALSE, 5);
+                        gtk_container_set_border_width(GTK_CONTAINER(vbox3), 5);
+                        vbox3.show();
+                        frame.add(vbox3);
+                        {
+                            auto table = ui::Table(3, 2, FALSE);
+                            table.show();
+                            vbox3.pack_start(table, TRUE, TRUE, 0);
+                            gtk_table_set_row_spacings(table, 5);
+                            gtk_table_set_col_spacings(table, 5);
+                            {
+                                auto label = ui::Label("Fixed");
+                                label.show();
                                 table.attach(label, {0, 1, 0, 1}, {GTK_EXPAND | GTK_FILL, 0});
-                                                       }
-                                                       {
-                                                               auto check = ui::CheckButton::from( gtk_check_button_new() );
-                                                               check.show();
+                            }
+                            {
+                                auto check = ui::CheckButton::from(gtk_check_button_new());
+                                check.show();
                                 table.attach(check, {1, 2, 0, 1}, {GTK_EXPAND | GTK_FILL, 0});
-                                                               m_subdivisions.m_enabled = check;
-                                                               guint handler_id = check.connect( "toggled", G_CALLBACK( &Subdivisions::applyGtk ), &m_subdivisions );
-                                                               g_object_set_data( G_OBJECT( check ), "handler", gint_to_pointer( handler_id ) );
-                                                       }
-                                                       {
-                                                               auto label = ui::Label( "Horizontal" );
-                                                               label.show();
+                                m_subdivisions.m_enabled = check;
+                                guint handler_id = check.connect("toggled", G_CALLBACK(&Subdivisions::applyGtk),
+                                                                 &m_subdivisions);
+                                g_object_set_data(G_OBJECT(check), "handler", gint_to_pointer(handler_id));
+                            }
+                            {
+                                auto label = ui::Label("Horizontal");
+                                label.show();
                                 table.attach(label, {0, 1, 1, 2}, {GTK_EXPAND | GTK_FILL, 0});
-                                                       }
-                                                       {
-                                                               auto entry = ui::Entry(ui::New);
-                                                               entry.show();
+                            }
+                            {
+                                auto entry = ui::Entry(ui::New);
+                                entry.show();
                                 table.attach(entry, {1, 2, 1, 2}, {GTK_EXPAND | GTK_FILL, 0});
-                                                               m_subdivisions.m_horizontal = entry;
-                                                               m_horizontalSubdivisionsEntry.connect( entry );
-                                                       }
-                                                       {
-                                                               auto label = ui::Label( "Vertical" );
-                                                               label.show();
+                                m_subdivisions.m_horizontal = entry;
+                                m_horizontalSubdivisionsEntry.connect(entry);
+                            }
+                            {
+                                auto label = ui::Label("Vertical");
+                                label.show();
                                 table.attach(label, {0, 1, 2, 3}, {GTK_EXPAND | GTK_FILL, 0});
-                                                       }
-                                                       {
-                                                               auto entry = ui::Entry(ui::New);
-                                                               entry.show();
+                            }
+                            {
+                                auto entry = ui::Entry(ui::New);
+                                entry.show();
                                 table.attach(entry, {1, 2, 2, 3}, {GTK_EXPAND | GTK_FILL, 0});
-                                                               m_subdivisions.m_vertical = entry;
-                                                               m_verticalSubdivisionsEntry.connect( entry );
-                                                       }
-                                               }
-                                       }
-                               }
-                       }
-                       {
-                               auto frame = ui::Frame( "Texturing" );
-                               frame.show();
-                               hbox.pack_start( frame, TRUE, TRUE, 0 );
-                               {
-                                       auto vbox2 = ui::VBox( FALSE, 5 );
-                                       vbox2.show();
-                                       frame.add(vbox2);
-                                       gtk_container_set_border_width( GTK_CONTAINER( vbox2 ), 5 );
-                                       {
-                                               auto label = ui::Label( "Name:" );
-                                               label.show();
-                                               vbox2.pack_start( label, TRUE, TRUE, 0 );
-                                               gtk_label_set_justify( label, GTK_JUSTIFY_LEFT );
-                                               gtk_misc_set_alignment( GTK_MISC( label ), 0, 0.5 );
-                                       }
-                                       {
-                                               auto entry = ui::Entry(ui::New);
-                                               //  gtk_editable_set_editable (GTK_ENTRY (entry), false);
-                                               entry.show();
-                                               vbox2.pack_start( entry, TRUE, TRUE, 0 );
-                                               AddDialogData( entry, m_strName );
-
-                                               entry.connect( "key_press_event", G_CALLBACK( OnDialogKey ), 0 );
-                                       }
-                                       {
-                                               auto table = ui::Table( 5, 4, FALSE );
-                                               table.show();
-                                               vbox2.pack_start( table, TRUE, TRUE, 0 );
-                                               gtk_table_set_row_spacings( table, 5 );
-                                               gtk_table_set_col_spacings( table, 5 );
-                                               {
-                                                       auto label = ui::Label( "Horizontal Shift Step" );
-                                                       label.show();
+                                m_subdivisions.m_vertical = entry;
+                                m_verticalSubdivisionsEntry.connect(entry);
+                            }
+                        }
+                    }
+                }
+            }
+            {
+                auto frame = ui::Frame("Texturing");
+                frame.show();
+                hbox.pack_start(frame, TRUE, TRUE, 0);
+                {
+                    auto vbox2 = ui::VBox(FALSE, 5);
+                    vbox2.show();
+                    frame.add(vbox2);
+                    gtk_container_set_border_width(GTK_CONTAINER(vbox2), 5);
+                    {
+                        auto label = ui::Label("Name:");
+                        label.show();
+                        vbox2.pack_start(label, TRUE, TRUE, 0);
+                        gtk_label_set_justify(label, GTK_JUSTIFY_LEFT);
+                        gtk_misc_set_alignment(GTK_MISC(label), 0, 0.5);
+                    }
+                    {
+                        auto entry = ui::Entry(ui::New);
+                        //  gtk_editable_set_editable (GTK_ENTRY (entry), false);
+                        entry.show();
+                        vbox2.pack_start(entry, TRUE, TRUE, 0);
+                        AddDialogData(entry, m_strName);
+
+                        entry.connect("key_press_event", G_CALLBACK(OnDialogKey), 0);
+                    }
+                    {
+                        auto table = ui::Table(5, 4, FALSE);
+                        table.show();
+                        vbox2.pack_start(table, TRUE, TRUE, 0);
+                        gtk_table_set_row_spacings(table, 5);
+                        gtk_table_set_col_spacings(table, 5);
+                        {
+                            auto label = ui::Label("Horizontal Shift Step");
+                            label.show();
                             table.attach(label, {2, 4, 0, 1}, {GTK_FILL | GTK_EXPAND, 0});
-                                                       gtk_misc_set_alignment( GTK_MISC( label ), 0, 0.5 );
-                                               }
-                                               {
-                                                       auto label = ui::Label( "Vertical Shift Step" );
-                                                       label.show();
+                            gtk_misc_set_alignment(GTK_MISC(label), 0, 0.5);
+                        }
+                        {
+                            auto label = ui::Label("Vertical Shift Step");
+                            label.show();
                             table.attach(label, {2, 4, 1, 2}, {GTK_FILL | GTK_EXPAND, 0});
-                                                       gtk_misc_set_alignment( GTK_MISC( label ), 0, 0.5 );
-                                               }
-                                               {
-                                                       auto label = ui::Label( "Horizontal Stretch Step" );
-                                                       label.show();
+                            gtk_misc_set_alignment(GTK_MISC(label), 0, 0.5);
+                        }
+                        {
+                            auto label = ui::Label("Horizontal Stretch Step");
+                            label.show();
                             table.attach(label, {2, 3, 2, 3}, {GTK_FILL | GTK_EXPAND, 0});
-                                                       gtk_misc_set_alignment( GTK_MISC( label ), 0, 0.5 );
-                                               }
-                                               {
-                                                       auto button = ui::Button( "Flip" );
-                                                       button.show();
+                            gtk_misc_set_alignment(GTK_MISC(label), 0, 0.5);
+                        }
+                        {
+                            auto button = ui::Button("Flip");
+                            button.show();
                             table.attach(button, {3, 4, 2, 3}, {GTK_FILL, 0});
-                                                       button.connect( "clicked", G_CALLBACK( OnBtnPatchFlipX ), 0 );
-                                                       button.dimensions(60, -1);
-                                               }
-                                               {
-                                                       auto label = ui::Label( "Vertical Stretch Step" );
-                                                       label.show();
+                            button.connect("clicked", G_CALLBACK(OnBtnPatchFlipX), 0);
+                            button.dimensions(60, -1);
+                        }
+                        {
+                            auto label = ui::Label("Vertical Stretch Step");
+                            label.show();
                             table.attach(label, {2, 3, 3, 4}, {GTK_FILL | GTK_EXPAND, 0});
-                                                       gtk_misc_set_alignment( GTK_MISC( label ), 0, 0.5 );
-                                               }
-                                               {
-                                                       auto button = ui::Button( "Flip" );
-                                                       button.show();
+                            gtk_misc_set_alignment(GTK_MISC(label), 0, 0.5);
+                        }
+                        {
+                            auto button = ui::Button("Flip");
+                            button.show();
                             table.attach(button, {3, 4, 3, 4}, {GTK_FILL, 0});
-                                                       button.connect( "clicked", G_CALLBACK( OnBtnPatchFlipY ), 0 );
-                                                       button.dimensions(60, -1);
-                                               }
-                                               {
-                                                       auto label = ui::Label( "Rotate Step" );
-                                                       label.show();
+                            button.connect("clicked", G_CALLBACK(OnBtnPatchFlipY), 0);
+                            button.dimensions(60, -1);
+                        }
+                        {
+                            auto label = ui::Label("Rotate Step");
+                            label.show();
                             table.attach(label, {2, 4, 4, 5}, {GTK_FILL | GTK_EXPAND, 0});
-                                                       gtk_misc_set_alignment( GTK_MISC( label ), 0, 0.5 );
-                                               }
-                                               {
-                                                       auto entry = ui::Entry(ui::New);
-                                                       entry.show();
+                            gtk_misc_set_alignment(GTK_MISC(label), 0, 0.5);
+                        }
+                        {
+                            auto entry = ui::Entry(ui::New);
+                            entry.show();
                             table.attach(entry, {0, 1, 0, 1}, {GTK_FILL, 0});
-                                                       entry.dimensions(50, -1);
-                                                       g_object_set_data( G_OBJECT( window ), "hshift_entry", (void *) entry );
-                                                       // we fill in this data, if no patch is selected the widgets are unmodified when the inspector is raised
-                                                       // so we need to have at least one initialisation somewhere
-                                                       entry_set_float( entry, g_pi_globals.shift[0] );
-
-                                                       auto adj = ui::Adjustment( 0, -8192, 8192, 1, 1, 0 );
-                                                       adj.connect( "value_changed", G_CALLBACK( OnSpinChanged ), (gpointer) entry );
-                                                       g_object_set_data( G_OBJECT( window ), "hshift_adj", (gpointer) adj );
-
-                                                       auto spin = ui::SpinButton( adj, 1, 0 );
-                                                       spin.show();
+                            entry.dimensions(50, -1);
+                            g_object_set_data(G_OBJECT(window), "hshift_entry", (void *) entry);
+                            // we fill in this data, if no patch is selected the widgets are unmodified when the inspector is raised
+                            // so we need to have at least one initialisation somewhere
+                            entry_set_float(entry, g_pi_globals.shift[0]);
+
+                            auto adj = ui::Adjustment(0, -8192, 8192, 1, 1, 0);
+                            adj.connect("value_changed", G_CALLBACK(OnSpinChanged), (gpointer) entry);
+                            g_object_set_data(G_OBJECT(window), "hshift_adj", (gpointer) adj);
+
+                            auto spin = ui::SpinButton(adj, 1, 0);
+                            spin.show();
                             table.attach(spin, {1, 2, 0, 1}, {0, 0});
-                                                       spin.dimensions(10, -1);
-                                                       gtk_widget_set_can_focus( spin, false );
-                                               }
-                                               {
-                                                       auto entry = ui::Entry(ui::New);
-                                                       entry.show();
+                            spin.dimensions(10, -1);
+                            gtk_widget_set_can_focus(spin, false);
+                        }
+                        {
+                            auto entry = ui::Entry(ui::New);
+                            entry.show();
                             table.attach(entry, {0, 1, 1, 2}, {GTK_FILL, 0});
-                                                       entry.dimensions(50, -1);
-                                                       entry_set_float( entry, g_pi_globals.shift[1] );
+                            entry.dimensions(50, -1);
+                            entry_set_float(entry, g_pi_globals.shift[1]);
 
-                                                       auto adj = ui::Adjustment( 0, -8192, 8192, 1, 1, 0 );
-                                                       adj.connect( "value_changed", G_CALLBACK( OnSpinChanged ), entry );
-                                                       g_object_set_data( G_OBJECT( window ), "vshift_adj", (gpointer) adj );
+                            auto adj = ui::Adjustment(0, -8192, 8192, 1, 1, 0);
+                            adj.connect("value_changed", G_CALLBACK(OnSpinChanged), entry);
+                            g_object_set_data(G_OBJECT(window), "vshift_adj", (gpointer) adj);
 
-                                                       auto spin = ui::SpinButton( adj, 1, 0 );
-                                                       spin.show();
+                            auto spin = ui::SpinButton(adj, 1, 0);
+                            spin.show();
                             table.attach(spin, {1, 2, 1, 2}, {0, 0});
-                                                       spin.dimensions(10, -1);
-                                                       gtk_widget_set_can_focus( spin, false );
-                                               }
-                                               {
-                                                       auto entry = ui::Entry(ui::New);
-                                                       entry.show();
+                            spin.dimensions(10, -1);
+                            gtk_widget_set_can_focus(spin, false);
+                        }
+                        {
+                            auto entry = ui::Entry(ui::New);
+                            entry.show();
                             table.attach(entry, {0, 1, 2, 3}, {GTK_FILL, 0});
-                                                       entry.dimensions(50, -1);
-                                                       entry_set_float( entry, g_pi_globals.scale[0] );
+                            entry.dimensions(50, -1);
+                            entry_set_float(entry, g_pi_globals.scale[0]);
 
-                                                       auto adj = ui::Adjustment( 0, -1000, 1000, 1, 1, 0 );
-                                                       adj.connect( "value_changed", G_CALLBACK( OnSpinChanged ), entry );
-                                                       g_object_set_data( G_OBJECT( window ), "hscale_adj", (gpointer) adj );
+                            auto adj = ui::Adjustment(0, -1000, 1000, 1, 1, 0);
+                            adj.connect("value_changed", G_CALLBACK(OnSpinChanged), entry);
+                            g_object_set_data(G_OBJECT(window), "hscale_adj", (gpointer) adj);
 
-                                                       auto spin = ui::SpinButton( adj, 1, 0 );
-                                                       spin.show();
+                            auto spin = ui::SpinButton(adj, 1, 0);
+                            spin.show();
                             table.attach(spin, {1, 2, 2, 3}, {0, 0});
-                                                       spin.dimensions(10, -1);
-                                                       gtk_widget_set_can_focus( spin, false );
-                                               }
-                                               {
-                                                       auto entry = ui::Entry(ui::New);
-                                                       entry.show();
+                            spin.dimensions(10, -1);
+                            gtk_widget_set_can_focus(spin, false);
+                        }
+                        {
+                            auto entry = ui::Entry(ui::New);
+                            entry.show();
                             table.attach(entry, {0, 1, 3, 4}, {GTK_FILL, 0});
-                                                       entry.dimensions(50, -1);
-                                                       entry_set_float( entry, g_pi_globals.scale[1] );
+                            entry.dimensions(50, -1);
+                            entry_set_float(entry, g_pi_globals.scale[1]);
 
-                                                       auto adj = ui::Adjustment( 0, -1000, 1000, 1, 1, 0 );
-                                                       adj.connect( "value_changed", G_CALLBACK( OnSpinChanged ), entry );
-                                                       g_object_set_data( G_OBJECT( window ), "vscale_adj", (gpointer) adj );
+                            auto adj = ui::Adjustment(0, -1000, 1000, 1, 1, 0);
+                            adj.connect("value_changed", G_CALLBACK(OnSpinChanged), entry);
+                            g_object_set_data(G_OBJECT(window), "vscale_adj", (gpointer) adj);
 
-                                                       auto spin = ui::SpinButton( adj, 1, 0 );
-                                                       spin.show();
+                            auto spin = ui::SpinButton(adj, 1, 0);
+                            spin.show();
                             table.attach(spin, {1, 2, 3, 4}, {0, 0});
-                                                       spin.dimensions(10, -1);
-                                                       gtk_widget_set_can_focus( spin, false );
-                                               }
-                                               {
-                                                       auto entry = ui::Entry(ui::New);
-                                                       entry.show();
+                            spin.dimensions(10, -1);
+                            gtk_widget_set_can_focus(spin, false);
+                        }
+                        {
+                            auto entry = ui::Entry(ui::New);
+                            entry.show();
                             table.attach(entry, {0, 1, 4, 5}, {GTK_FILL, 0});
-                                                       entry.dimensions(50, -1);
-                                                       entry_set_float( entry, g_pi_globals.rotate );
+                            entry.dimensions(50, -1);
+                            entry_set_float(entry, g_pi_globals.rotate);
 
-                                                       auto adj = ui::Adjustment( 0, -1000, 1000, 1, 1, 0 ); // NOTE: Arnout - this really should be 360 but can't change it anymore as it could break existing maps
-                                                       adj.connect( "value_changed", G_CALLBACK( OnSpinChanged ), entry );
-                                                       g_object_set_data( G_OBJECT( window ), "rotate_adj", (gpointer) adj );
+                            auto adj = ui::Adjustment(0, -1000, 1000, 1, 1,
+                                                      0); // NOTE: Arnout - this really should be 360 but can't change it anymore as it could break existing maps
+                            adj.connect("value_changed", G_CALLBACK(OnSpinChanged), entry);
+                            g_object_set_data(G_OBJECT(window), "rotate_adj", (gpointer) adj);
 
-                                                       auto spin = ui::SpinButton( adj, 1, 0 );
-                                                       spin.show();
+                            auto spin = ui::SpinButton(adj, 1, 0);
+                            spin.show();
                             table.attach(spin, {1, 2, 4, 5}, {0, 0});
-                                                       spin.dimensions(10, -1);
-                                                       gtk_widget_set_can_focus( spin, false );
-                                               }
-                                       }
-                                       auto hbox2 = ui::HBox( TRUE, 5 );
-                                       hbox2.show();
-                                       vbox2.pack_start( hbox2, TRUE, FALSE, 0 );
-                                       {
-                                               auto button = ui::Button( "Auto Cap" );
-                                               button.show();
-                                               hbox2.pack_end(button, TRUE, FALSE, 0);
-                                               button.connect( "clicked", G_CALLBACK( OnBtnPatchAutoCap ), 0 );
-                                               button.dimensions(60, -1);
-                                       }
-                                       {
-                                               auto button = ui::Button( "CAP" );
-                                               button.show();
-                                               hbox2.pack_end(button, TRUE, FALSE, 0);
-                                               button.connect( "clicked", G_CALLBACK( OnBtnPatchdetails ), 0 );
-                                               button.dimensions(60, -1);
-                                       }
-                                       {
-                                               auto button = ui::Button( "Set..." );
-                                               button.show();
-                                               hbox2.pack_end(button, TRUE, FALSE, 0);
-                                               button.connect( "clicked", G_CALLBACK( OnBtnPatchreset ), 0 );
-                                               button.dimensions(60, -1);
-                                       }
-                                       {
-                                               auto button = ui::Button( "Natural" );
-                                               button.show();
-                                               hbox2.pack_end(button, TRUE, FALSE, 0);
-                                               button.connect( "clicked", G_CALLBACK( OnBtnPatchnatural ), 0 );
-                                               button.dimensions(60, -1);
-                                       }
-                                       {
-                                               auto button = ui::Button( "Fit" );
-                                               button.show();
-                                               hbox2.pack_end(button, TRUE, FALSE, 0);
-                                               button.connect( "clicked", G_CALLBACK( OnBtnPatchfit ), 0 );
-                                               button.dimensions(60, -1);
-                                       }
-                               }
-                       }
-               }
-       }
-
-       return window;
+                            spin.dimensions(10, -1);
+                            gtk_widget_set_can_focus(spin, false);
+                        }
+                    }
+                    auto hbox2 = ui::HBox(TRUE, 5);
+                    hbox2.show();
+                    vbox2.pack_start(hbox2, TRUE, FALSE, 0);
+                    {
+                        auto button = ui::Button("Auto Cap");
+                        button.show();
+                        hbox2.pack_end(button, TRUE, FALSE, 0);
+                        button.connect("clicked", G_CALLBACK(OnBtnPatchAutoCap), 0);
+                        button.dimensions(60, -1);
+                    }
+                    {
+                        auto button = ui::Button("CAP");
+                        button.show();
+                        hbox2.pack_end(button, TRUE, FALSE, 0);
+                        button.connect("clicked", G_CALLBACK(OnBtnPatchdetails), 0);
+                        button.dimensions(60, -1);
+                    }
+                    {
+                        auto button = ui::Button("Set...");
+                        button.show();
+                        hbox2.pack_end(button, TRUE, FALSE, 0);
+                        button.connect("clicked", G_CALLBACK(OnBtnPatchreset), 0);
+                        button.dimensions(60, -1);
+                    }
+                    {
+                        auto button = ui::Button("Natural");
+                        button.show();
+                        hbox2.pack_end(button, TRUE, FALSE, 0);
+                        button.connect("clicked", G_CALLBACK(OnBtnPatchnatural), 0);
+                        button.dimensions(60, -1);
+                    }
+                    {
+                        auto button = ui::Button("Fit");
+                        button.show();
+                        hbox2.pack_end(button, TRUE, FALSE, 0);
+                        button.connect("clicked", G_CALLBACK(OnBtnPatchfit), 0);
+                        button.dimensions(60, -1);
+                    }
+                }
+            }
+        }
+    }
+
+    return window;
 }
 
 // sync the dialog our internal data structures
-void PatchInspector::exportData(){
-       m_bListenChanged = false;
-       Dialog::exportData();
-       m_bListenChanged = true;
+void PatchInspector::exportData()
+{
+    m_bListenChanged = false;
+    Dialog::exportData();
+    m_bListenChanged = true;
 }
-void PatchInspector::importData(){
-       m_bListenChanged = false;
-       Dialog::importData();
-       m_bListenChanged = true;
+
+void PatchInspector::importData()
+{
+    m_bListenChanged = false;
+    Dialog::importData();
+    m_bListenChanged = true;
 }
 
 // read the map and feed in the stuff to the dialog box
-void PatchInspector::GetPatchInfo(){
-       if ( g_pGameDescription->mGameType == "doom3" ) {
-               m_subdivisions.update();
-       }
-
-       if ( GlobalSelectionSystem().countSelected() == 0 ) {
-               m_Patch = 0;
-       }
-       else
-       {
-               m_Patch = Node_getPatch( GlobalSelectionSystem().ultimateSelected().path().top() );
-       }
-
-       if ( m_Patch != 0 ) {
-               m_strName = m_Patch->GetShader();
-
-               // fill in the numbers for Row / Col selection
-               m_bListenChanged = false;
-
-               {
-                       gtk_combo_box_set_active( m_pRowCombo, 0 );
-
-                       for ( std::size_t i = 0; i < m_countRows; ++i )
-                       {
-                               gtk_combo_box_text_remove( m_pRowCombo, gint( m_countRows - i - 1 ) );
-                       }
-
-                       m_countRows = m_Patch->getHeight();
-                       for ( std::size_t i = 0; i < m_countRows; ++i )
-                       {
-                               char buffer[16];
-                               sprintf( buffer, "%u", Unsigned( i ) );
-                               gtk_combo_box_text_append_text( m_pRowCombo, buffer );
-                       }
-
-                       gtk_combo_box_set_active( m_pRowCombo, 0 );
-               }
-
-               {
-                       gtk_combo_box_set_active( m_pColCombo, 0 );
-
-                       for ( std::size_t i = 0; i < m_countCols; ++i )
-                       {
-                               gtk_combo_box_text_remove( m_pColCombo, gint( m_countCols - i - 1 ) );
-                       }
-
-                       m_countCols = m_Patch->getWidth();
-                       for ( std::size_t i = 0; i < m_countCols; ++i )
-                       {
-                               char buffer[16];
-                               sprintf( buffer, "%u", Unsigned( i ) );
-                               gtk_combo_box_text_append_text( m_pColCombo, buffer );
-                       }
-
-                       gtk_combo_box_set_active( m_pColCombo, 0 );
-               }
-
-               m_bListenChanged = true;
-
-       }
-       else
-       {
-               //globalOutputStream() << "WARNING: no patch\n";
-       }
-       // fill in our internal structs
-       m_nRow = 0; m_nCol = 0;
-       UpdateRowColInfo();
-       // now update the dialog box
-       importData();
+void PatchInspector::GetPatchInfo()
+{
+    if (g_pGameDescription->mGameType == "doom3") {
+        m_subdivisions.update();
+    }
+
+    if (GlobalSelectionSystem().countSelected() == 0) {
+        m_Patch = 0;
+    } else {
+        m_Patch = Node_getPatch(GlobalSelectionSystem().ultimateSelected().path().top());
+    }
+
+    if (m_Patch != 0) {
+        m_strName = m_Patch->GetShader();
+
+        // fill in the numbers for Row / Col selection
+        m_bListenChanged = false;
+
+        {
+            gtk_combo_box_set_active(m_pRowCombo, 0);
+
+            for (std::size_t i = 0; i < m_countRows; ++i) {
+                gtk_combo_box_text_remove(m_pRowCombo, gint(m_countRows - i - 1));
+            }
+
+            m_countRows = m_Patch->getHeight();
+            for (std::size_t i = 0; i < m_countRows; ++i) {
+                char buffer[16];
+                sprintf(buffer, "%u", Unsigned(i));
+                gtk_combo_box_text_append_text(m_pRowCombo, buffer);
+            }
+
+            gtk_combo_box_set_active(m_pRowCombo, 0);
+        }
+
+        {
+            gtk_combo_box_set_active(m_pColCombo, 0);
+
+            for (std::size_t i = 0; i < m_countCols; ++i) {
+                gtk_combo_box_text_remove(m_pColCombo, gint(m_countCols - i - 1));
+            }
+
+            m_countCols = m_Patch->getWidth();
+            for (std::size_t i = 0; i < m_countCols; ++i) {
+                char buffer[16];
+                sprintf(buffer, "%u", Unsigned(i));
+                gtk_combo_box_text_append_text(m_pColCombo, buffer);
+            }
+
+            gtk_combo_box_set_active(m_pColCombo, 0);
+        }
+
+        m_bListenChanged = true;
+
+    } else {
+        //globalOutputStream() << "WARNING: no patch\n";
+    }
+    // fill in our internal structs
+    m_nRow = 0;
+    m_nCol = 0;
+    UpdateRowColInfo();
+    // now update the dialog box
+    importData();
 }
 
 // read the current patch on map and initialize m_fX m_fY accordingly
 // NOTE: don't call UpdateData in there, it's not meant for
-void PatchInspector::UpdateRowColInfo(){
-       m_fX = m_fY = m_fZ = m_fS = m_fT = 0.0;
-
-       if ( m_Patch != 0 ) {
-               // we rely on whatever active row/column has been set before we get called
-               std::size_t r = m_nRow;
-               std::size_t c = m_nCol;
-               if ( r < m_Patch->getHeight()
-                        && c < m_Patch->getWidth() ) {
-                       const PatchControl& p = m_Patch->ctrlAt( r,c );
-                       m_fX = p.m_vertex[0];
-                       m_fY = p.m_vertex[1];
-                       m_fZ = p.m_vertex[2];
-                       m_fS = p.m_texcoord[0];
-                       m_fT = p.m_texcoord[1];
-               }
-       }
+void PatchInspector::UpdateRowColInfo()
+{
+    m_fX = m_fY = m_fZ = m_fS = m_fT = 0.0;
+
+    if (m_Patch != 0) {
+        // we rely on whatever active row/column has been set before we get called
+        std::size_t r = m_nRow;
+        std::size_t c = m_nCol;
+        if (r < m_Patch->getHeight()
+            && c < m_Patch->getWidth()) {
+            const PatchControl &p = m_Patch->ctrlAt(r, c);
+            m_fX = p.m_vertex[0];
+            m_fY = p.m_vertex[1];
+            m_fZ = p.m_vertex[2];
+            m_fS = p.m_texcoord[0];
+            m_fT = p.m_texcoord[1];
+        }
+    }
 }
 
 
-void PatchInspector_SelectionChanged( const Selectable& selectable ){
-       PatchInspector_queueDraw();
+void PatchInspector_SelectionChanged(const Selectable &selectable)
+{
+    PatchInspector_queueDraw();
 }
 
 
 #include "preferencesystem.h"
 
 
-void PatchInspector_Construct(){
-       GlobalCommands_insert( "PatchInspector", makeCallbackF(PatchInspector_toggleShown), Accelerator( 'S', (GdkModifierType)GDK_SHIFT_MASK ) );
-
-       GlobalPreferenceSystem().registerPreference( "PatchWnd", make_property<WindowPositionTracker_String>( g_PatchInspector.m_position_tracker ) );
-       GlobalPreferenceSystem().registerPreference( "SI_PatchTexdef_Scale1", make_property_string( g_pi_globals.scale[0] ) );
-       GlobalPreferenceSystem().registerPreference( "SI_PatchTexdef_Scale2", make_property_string( g_pi_globals.scale[1] ) );
-       GlobalPreferenceSystem().registerPreference( "SI_PatchTexdef_Shift1", make_property_string( g_pi_globals.shift[0] ) );
-       GlobalPreferenceSystem().registerPreference( "SI_PatchTexdef_Shift2", make_property_string( g_pi_globals.shift[1] ) );
-       GlobalPreferenceSystem().registerPreference( "SI_PatchTexdef_Rotate", make_property_string( g_pi_globals.rotate ) );
-
-       typedef FreeCaller<void(const Selectable&), PatchInspector_SelectionChanged> PatchInspectorSelectionChangedCaller;
-       GlobalSelectionSystem().addSelectionChangeCallback( PatchInspectorSelectionChangedCaller() );
-       typedef FreeCaller<void(), PatchInspector_queueDraw> PatchInspectorQueueDrawCaller;
-       Patch_addTextureChangedCallback( PatchInspectorQueueDrawCaller() );
+void PatchInspector_Construct()
+{
+    GlobalCommands_insert("PatchInspector", makeCallbackF(PatchInspector_toggleShown),
+                          Accelerator('S', (GdkModifierType) GDK_SHIFT_MASK));
+
+    GlobalPreferenceSystem().registerPreference("PatchWnd", make_property<WindowPositionTracker_String>(
+            g_PatchInspector.m_position_tracker));
+    GlobalPreferenceSystem().registerPreference("SI_PatchTexdef_Scale1", make_property_string(g_pi_globals.scale[0]));
+    GlobalPreferenceSystem().registerPreference("SI_PatchTexdef_Scale2", make_property_string(g_pi_globals.scale[1]));
+    GlobalPreferenceSystem().registerPreference("SI_PatchTexdef_Shift1", make_property_string(g_pi_globals.shift[0]));
+    GlobalPreferenceSystem().registerPreference("SI_PatchTexdef_Shift2", make_property_string(g_pi_globals.shift[1]));
+    GlobalPreferenceSystem().registerPreference("SI_PatchTexdef_Rotate", make_property_string(g_pi_globals.rotate));
+
+    typedef FreeCaller<void(const Selectable &), PatchInspector_SelectionChanged> PatchInspectorSelectionChangedCaller;
+    GlobalSelectionSystem().addSelectionChangeCallback(PatchInspectorSelectionChangedCaller());
+    typedef FreeCaller<void(), PatchInspector_queueDraw> PatchInspectorQueueDrawCaller;
+    Patch_addTextureChangedCallback(PatchInspectorQueueDrawCaller());
 }
-void PatchInspector_Destroy(){
+
+void PatchInspector_Destroy()
+{
 }