From 056cd3cdfc6b6ea89643454059e7e1538b3002ea Mon Sep 17 00:00:00 2001
From: David Capello <david@igarastudio.com>
Date: Tue, 5 May 2020 20:10:54 -0300
Subject: [PATCH] Minor changes in ColorShades member functions to avoid
 copying a Shade object

---
 src/app/ui/color_shades.cpp | 10 ----------
 src/app/ui/color_shades.h   |  4 ++--
 2 files changed, 2 insertions(+), 12 deletions(-)

diff --git a/src/app/ui/color_shades.cpp b/src/app/ui/color_shades.cpp
index 598b64176..2f4e1e50a 100644
--- a/src/app/ui/color_shades.cpp
+++ b/src/app/ui/color_shades.cpp
@@ -84,16 +84,6 @@ doc::Remap* ColorShades::createShadeRemap(bool left)
   return remap.release();
 }
 
-int ColorShades::size() const
-{
-  return int(m_shade.size());
-}
-
-Shade ColorShades::getShade() const
-{
-  return m_shade;
-}
-
 void ColorShades::setShade(const Shade& shade)
 {
   m_shade = shade;
diff --git a/src/app/ui/color_shades.h b/src/app/ui/color_shades.h
index 7917e4439..aea218056 100644
--- a/src/app/ui/color_shades.h
+++ b/src/app/ui/color_shades.h
@@ -35,9 +35,9 @@ namespace app {
     void setMinColors(int minColors);
     void reverseShadeColors();
     doc::Remap* createShadeRemap(bool left);
-    int size() const;
+    int size() const { return int(m_shade.size()); }
 
-    Shade getShade() const;
+    const Shade& getShade() const { return m_shade; }
     void setShade(const Shade& shade);
 
     int getHotEntry() const { return m_hotIndex; }