common: Functions that can declare const are refactored to use const
authorJunsuChoi <jsuya.choi@samsung.com>
Fri, 19 Feb 2021 06:22:17 +0000 (15:22 +0900)
committerJunsuChoi <jsuya.choi@samsung.com>
Fri, 19 Feb 2021 08:34:58 +0000 (17:34 +0900)
Member functions that don't mutate their objects should be declared "const"

src/lib/sw_engine/tvgSwCommon.h
src/lib/sw_engine/tvgSwRenderer.cpp
src/lib/tvgPictureImpl.h
src/lib/tvgSceneImpl.h
src/lib/tvgShapeImpl.h

index e06d378..82a6023 100644 (file)
@@ -75,13 +75,13 @@ struct SwPoint
         return (x != rhs.x || y != rhs.y);
     }
 
-    bool zero()
+    bool zero() const
     {
         if (x == 0 && y == 0) return true;
         else return false;
     }
 
-    bool small()
+    bool small() const
     {
         //2 is epsilon...
         if (abs(x) < 2 && abs(y) < 2) return true;
index f5cab79..09b199b 100644 (file)
@@ -40,7 +40,7 @@ struct SwTask : Task
     uint32_t opacity;
     SwBBox bbox = {{0, 0}, {0, 0}};       //Whole Rendering Region
 
-    void bounds(uint32_t* x, uint32_t* y, uint32_t* w, uint32_t* h)
+    void bounds(uint32_t* x, uint32_t* y, uint32_t* w, uint32_t* h) const
     {
         //Range over?
         auto xx = bbox.min.x > 0 ? bbox.min.x : 0;
index d39809d..e3b0c5d 100644 (file)
@@ -130,7 +130,7 @@ struct Picture::Impl
         return false;
     }
 
-    bool viewbox(float* x, float* y, float* w, float* h)
+    bool viewbox(float* x, float* y, float* w, float* h) const
     {
         if (!loader) return false;
         if (x) *x = loader->vx;
@@ -148,7 +148,7 @@ struct Picture::Impl
         return true;
     }
 
-    bool bounds(float* x, float* y, float* w, float* h)
+    bool bounds(float* x, float* y, float* w, float* h) const
     {
         if (!paint) return false;
         return paint->pImpl->bounds(x, y, w, h);
index 5494565..4745da7 100644 (file)
@@ -81,7 +81,7 @@ struct Scene::Impl
         return true;
     }
 
-    bool bounds(RenderMethod& renderer, uint32_t* px, uint32_t* py, uint32_t* pw, uint32_t* ph)
+    bool bounds(RenderMethod& renderer, uint32_t* px, uint32_t* py, uint32_t* pw, uint32_t* ph) const
     {
         if (paints.count == 0) return false;
 
@@ -113,7 +113,7 @@ struct Scene::Impl
         return true;
     }
 
-    bool bounds(float* px, float* py, float* pw, float* ph)
+    bool bounds(float* px, float* py, float* pw, float* ph) const
     {
         if (paints.count == 0) return false;
 
index 14f911e..c50f938 100644 (file)
@@ -168,7 +168,7 @@ struct ShapePath
         cmds[cmdCnt++] = PathCommand::Close;
     }
 
-    bool bounds(float* x, float* y, float* w, float* h)
+    bool bounds(float* x, float* y, float* w, float* h) const
     {
         if (ptsCnt == 0) return false;