Fix: Ensure ColorPickerTitlebar::PointerToScreen returns a value, Com…

…ment annoying qDebug
rurigk · Jul 19, 2020 · 05a8ce5 · 05a8ce5
1 parent 6d2b694
commit 05a8ce5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/colorpanel.cpp b/src/colorpanel.cpp
@@ -397,7 +397,7 @@ void ColorPanel::FillHistory()
         layout->addWidget(colorButton, row, column, Qt::AlignTop);
     }
 
-    qDebug() << "Row: " << row << " Column: " << historyColumns+1;
+    //qDebug() << "Row: " << row << " Column: " << historyColumns+1;
 
     QSpacerItem * spacer = new QSpacerItem(1, 1, QSizePolicy::Expanding, QSizePolicy::Expanding);
     layout->addItem(spacer, row, historyColumns+1, 1, -1, Qt::AlignTop);

diff --git a/src/colorpickertitlebar.cpp b/src/colorpickertitlebar.cpp
@@ -77,4 +77,5 @@ int ColorPickerTitlebar::PointerToScreen(QPoint pointer)
             return sIndex;
         }
     }
+    return 0;
 }
-Original file line number
+Diff line change
@@ Expand Up / @@ -77,4 +77,5 @@ int ColorPickerTitlebar::PointerToScreen(QPoint pointer) @@
                 return sIndex;
             }
         }
+        return 0;
     }