code cleanup: unused defines & some formatting.
authorCampbell Barton <ideasman42@gmail.com>
Thu, 8 Nov 2012 06:46:10 +0000 (06:46 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Thu, 8 Nov 2012 06:46:10 +0000 (06:46 +0000)
14 files changed:
intern/container/CTR_HashedPtr.h
source/blender/blenkernel/intern/cdderivedmesh.c
source/blender/bmesh/intern/bmesh_queries.c
source/blender/bmesh/operators/bmo_smooth_laplacian.c
source/blender/bmesh/operators/bmo_utils.c
source/blender/editors/mesh/editmesh_select.c
source/blender/imbuf/intern/dds/BlockDXT.h
source/blender/imbuf/intern/dds/Stream.h
source/blender/imbuf/intern/jp2.c
source/blender/imbuf/intern/radiance_hdr.c
source/blender/nodes/composite/nodes/node_composite_colorSpill.c
source/gameengine/Network/NG_NetworkDeviceInterface.h
source/gameengine/Rasterizer/RAS_ICanvas.h
source/gameengine/SceneGraph/SG_IObject.h

index b7ac460f2705e168e43501399156243a72a30c41..ee832eee153fc8d89bec1b95e0ca6b3e9323521d 100644 (file)
@@ -46,13 +46,13 @@ class CTR_HashedPtr
        void *m_valptr;
 public:
        CTR_HashedPtr(void *val) : m_valptr(val) {
-       };
+       }
        unsigned int hash() const {
                return CTR_Hash(m_valptr);
-       };
+       }
        inline friend bool operator ==(const CTR_HashedPtr & rhs, const CTR_HashedPtr & lhs) {
                return rhs.m_valptr == lhs.m_valptr;
-       };
+       }
        void *getValue() const {
                return m_valptr;
        }
index 2e0b3a3c64a5cd77bc4b91f48993758414f293f1..a0a402cc061f6ae22eefec068105a94c0a00f35d 100644 (file)
@@ -1956,12 +1956,11 @@ static DerivedMesh *cddm_from_bmesh_ex(struct BMesh *bm, int use_mdisps,
 
        /* avoid this where possiblem, takes extra memory */
        if (use_tessface) {
-               int *polyindex;
 
                BM_mesh_elem_index_ensure(bm, BM_FACE);
 
                index = dm->getTessFaceDataArray(dm, CD_ORIGINDEX);
-               for (i = 0; i < dm->numTessFaceData; i++, index++, polyindex++) {
+               for (i = 0; i < dm->numTessFaceData; i++, index++) {
                        MFace *mf = &mface[i];
                        const BMLoop **l = em_looptris[i];
                        efa = l[0]->f;
index 4b07dd74eef8078767abaea76fb9823defb35005..ed36a86274cafcddf42f06481cb83904c8caf01f 100644 (file)
@@ -401,6 +401,7 @@ BMLoop *BM_vert_step_fan_loop(BMLoop *l, BMEdge **e_step)
        }
        else {
                BLI_assert(0);
+               return NULL;
        }
 
        if (BM_edge_is_manifold(e_next)) {
index 51b9adb5de3048886d7f915549157116e1d85dbe..362123e412db161d1252fdcf2848c70231db0e41 100644 (file)
@@ -43,8 +43,8 @@
 
 #include "intern/bmesh_operators_private.h" /* own include */
 
-#define SMOOTH_LAPLACIAN_AREA_FACTOR 4.0f
-#define SMOOTH_LAPLACIAN_EDGE_FACTOR 2.0f
+// #define SMOOTH_LAPLACIAN_AREA_FACTOR 4.0f  /* UNUSED */
+// #define SMOOTH_LAPLACIAN_EDGE_FACTOR 2.0f  /* UNUSED */
 #define SMOOTH_LAPLACIAN_MAX_EDGE_PERCENTAGE 1.8f
 #define SMOOTH_LAPLACIAN_MIN_EDGE_PERCENTAGE 0.15f
 
index e311b383b861d90d49b02663767cc9ca70e5342b..aa514a5c0a77f53d500b290bf7a5ad734d487561 100644 (file)
@@ -282,7 +282,7 @@ void bmo_region_extend_exec(BMesh *bm, BMOperator *op)
 
 #define FACE_VIS       1
 #define FACE_FLAG      2
-#define FACE_MARK      4
+// #define FACE_MARK   4  /* UNUSED */
 #define FACE_FLIP      8
 
 /* NOTE: these are the original recalc_face_normals comment in editmesh_mods.c,
index ca4f0688f86180c9ee64eafc421558709cedca68..92ad0742ec95a84c670ddcabf87ea22c8158a7f7 100644 (file)
@@ -1199,8 +1199,8 @@ static float step_cost_3_v3(const float v1[3], const float v2[3], const float v3
 
 static float edgetag_cut_cost(BMEdge *e1, BMEdge *e2, BMVert *v)
 {
-       BMVert *v1 = (e1->v1 == v) ? e1->v2 : e1->v1;
-       BMVert *v2 = (e2->v1 == v) ? e2->v2 : e2->v1;
+       BMVert *v1 = BM_edge_other_vert(e1, v);
+       BMVert *v2 = BM_edge_other_vert(e2, v);
        return step_cost_3_v3(v1->co, v->co, v2->co);
 }
 
index 9b90b744b67ac67c66286735262faf26faaf96a1..0291816cd03f875aadb6aef788f4f9d80d0d3169 100644 (file)
@@ -173,24 +173,24 @@ struct AlphaBlockDXT5
        };
        */
        uint64 u;
-       uint8 alpha0() const { return u & 0xffLL; };
-       uint8 alpha1() const { return (u >> 8) & 0xffLL; };
-       uint8 bits0() const { return (u >> 16) & 0x7LL; };
-       uint8 bits1() const { return (u >> 19) & 0x7LL; };
-       uint8 bits2() const { return (u >> 22) & 0x7LL; };
-       uint8 bits3() const { return (u >> 25) & 0x7LL; };
-       uint8 bits4() const { return (u >> 28) & 0x7LL; };
-       uint8 bits5() const { return (u >> 31) & 0x7LL; };
-       uint8 bits6() const { return (u >> 34) & 0x7LL; };
-       uint8 bits7() const { return (u >> 37) & 0x7LL; };
-       uint8 bits8() const { return (u >> 40) & 0x7LL; };
-       uint8 bits9() const { return (u >> 43) & 0x7LL; };
-       uint8 bitsA() const { return (u >> 46) & 0x7LL; };
-       uint8 bitsB() const { return (u >> 49) & 0x7LL; };
-       uint8 bitsC() const { return (u >> 52) & 0x7LL; };
-       uint8 bitsD() const { return (u >> 55) & 0x7LL; };
-       uint8 bitsE() const { return (u >> 58) & 0x7LL; };
-       uint8 bitsF() const { return (u >> 61) & 0x7LL; };
+       uint8 alpha0() const { return u & 0xffLL; }
+       uint8 alpha1() const { return (u >> 8) & 0xffLL; }
+       uint8 bits0() const { return (u >> 16) & 0x7LL; }
+       uint8 bits1() const { return (u >> 19) & 0x7LL; }
+       uint8 bits2() const { return (u >> 22) & 0x7LL; }
+       uint8 bits3() const { return (u >> 25) & 0x7LL; }
+       uint8 bits4() const { return (u >> 28) & 0x7LL; }
+       uint8 bits5() const { return (u >> 31) & 0x7LL; }
+       uint8 bits6() const { return (u >> 34) & 0x7LL; }
+       uint8 bits7() const { return (u >> 37) & 0x7LL; }
+       uint8 bits8() const { return (u >> 40) & 0x7LL; }
+       uint8 bits9() const { return (u >> 43) & 0x7LL; }
+       uint8 bitsA() const { return (u >> 46) & 0x7LL; }
+       uint8 bitsB() const { return (u >> 49) & 0x7LL; }
+       uint8 bitsC() const { return (u >> 52) & 0x7LL; }
+       uint8 bitsD() const { return (u >> 55) & 0x7LL; }
+       uint8 bitsE() const { return (u >> 58) & 0x7LL; }
+       uint8 bitsF() const { return (u >> 61) & 0x7LL; }
        
        void evaluatePalette(uint8 alpha[8]) const;
        void evaluatePalette8(uint8 alpha[8]) const;
index 7fed4ca89e7291ade63d64cfa054a850c04df603..a1ac49b58da52f468126ad5968f0cfefe5f246ab 100644 (file)
@@ -35,7 +35,7 @@ struct Stream
        unsigned char *mem; // location in memory
        unsigned int size;  // size
        unsigned int pos;   // current position
-       Stream(unsigned char *m, unsigned int s) : mem(m), size(s), pos(0) {};
+       Stream(unsigned char *m, unsigned int s) : mem(m), size(s), pos(0) {}
        unsigned int seek(unsigned int p);
 };
 
index 692ba3c49ea7b9f62ee3ddf54495fc788a7aa9b8..d94f6368298426c7e6c3e66b13f4c39758d7d2c9 100644 (file)
@@ -41,7 +41,7 @@
 
 #include "openjpeg.h"
 
-#define JP2_FILEHEADER_SIZE 14
+// #define JP2_FILEHEADER_SIZE 14  /* UNUSED */
 
 static char JP2_HEAD[] = {0x0, 0x0, 0x0, 0x0C, 0x6A, 0x50, 0x20, 0x20, 0x0D, 0x0A, 0x87, 0x0A};
 static char J2K_HEAD[] = {0xFF, 0x4F, 0xFF, 0x51, 0x00};
index 72cec0e3749e86750841ef31a95703fce2a1e410..03ed1bb80086da86c7824459ab5cc31b37386080 100644 (file)
@@ -65,7 +65,6 @@
 #define BLU 2
 #define EXP 3
 #define COLXS 128
-#define STR_MAX 540
 typedef unsigned char RGBE[4];
 typedef float fCOLOR[3];
 
index e8965d50b47188dce7fe046742635ca7c8b2766d..44a5ac9e968c71c235c7308896a2b93d07ae79b9 100644 (file)
  *  \ingroup cmpnodes
  */
 
-
-
 #include "node_composite_util.h"
 
-#define AVG(a, b) ((a + b) / 2)
-
 /* ******************* Color Spill Supression ********************************* */
 static bNodeSocketTemplate cmp_node_color_spill_in[] = {
        {SOCK_RGBA, 1, N_("Image"), 1.0f, 1.0f, 1.0f, 1.0f},
@@ -49,6 +45,9 @@ static bNodeSocketTemplate cmp_node_color_spill_out[] = {
 
 #ifdef WITH_COMPOSITOR_LEGACY
 
+#define AVG(a, b) ((a + b) / 2)
+
+
 static void do_simple_spillmap_red(bNode *node, float* out, float *in)
 {
        NodeColorspill *ncs;
index 48edbdfc7fe7bd85ba135fd68d4fb2df1d51a367..6da478ecda5586fcbd13525314a10c5a5748475c 100644 (file)
@@ -43,8 +43,8 @@ private:
        // candidates for shared/common implementation class
        bool m_online;
 public:
-       NG_NetworkDeviceInterface() {};
-       virtual ~NG_NetworkDeviceInterface() {};
+       NG_NetworkDeviceInterface() {}
+       virtual ~NG_NetworkDeviceInterface() {}
 
        virtual void NextFrame()=0;
 
index 53195d797680ff1c546d1ba937410f99ebd41c9d..60b9f0520755e27a47052c096a0c8e62984efb22 100644 (file)
@@ -59,7 +59,7 @@ public:
        virtual 
        ~RAS_ICanvas(
        ) {
-       };
+       }
 
        virtual 
                void 
index 885eb8e1e90017b583f48255a95bf72f1001ba7e..6b57db1a467a49de970ac686e1b2a4abebdb8c54 100644 (file)
@@ -127,8 +127,8 @@ struct      SG_Callbacks
                m_schedulefunc(NULL),
                m_reschedulefunc(NULL)
        {
-       };
-               
+       }
+
        SG_Callbacks(
                SG_ReplicationNewCallback repfunc,
                SG_DestructionNewCallback destructfunc,
@@ -142,7 +142,7 @@ struct      SG_Callbacks
                m_schedulefunc(schedulefunc),
                m_reschedulefunc(reschedulefunc)
        {
-       };
+       }
 
        SG_ReplicationNewCallback       m_replicafunc;
        SG_DestructionNewCallback       m_destructionfunc;