cleanup: use const for smallhash & minor edits
authorCampbell Barton <ideasman42@gmail.com>
Thu, 18 Dec 2014 12:12:25 +0000 (13:12 +0100)
committerCampbell Barton <ideasman42@gmail.com>
Thu, 18 Dec 2014 12:12:25 +0000 (13:12 +0100)
source/blender/blenlib/BLI_smallhash.h
source/blender/blenlib/intern/edgehash.c
source/blender/blenlib/intern/smallhash.c
source/blender/editors/space_image/image_ops.c

index 5bc40a60515fa359a6ad36cac86c924663fb0174..e096354e5b1c4958d9c7d53ef448fab102097574 100644 (file)
@@ -52,7 +52,7 @@ typedef struct SmallHash {
 } SmallHash;
 
 typedef struct {
-       SmallHash *sh;
+       const SmallHash *sh;
        unsigned int i;
 } SmallHashIter;
 
@@ -63,14 +63,14 @@ void    BLI_smallhash_release(SmallHash *sh) ATTR_NONNULL(1);
 void    BLI_smallhash_insert(SmallHash *sh, uintptr_t key, void *item) ATTR_NONNULL(1);
 bool    BLI_smallhash_reinsert(SmallHash *sh, uintptr_t key, void *item) ATTR_NONNULL(1);
 bool    BLI_smallhash_remove(SmallHash *sh, uintptr_t key) ATTR_NONNULL(1);
-void   *BLI_smallhash_lookup(SmallHash *sh, uintptr_t key) ATTR_NONNULL(1) ATTR_WARN_UNUSED_RESULT;
-void  **BLI_smallhash_lookup_p(SmallHash *sh, uintptr_t key) ATTR_NONNULL(1) ATTR_WARN_UNUSED_RESULT;
-bool    BLI_smallhash_haskey(SmallHash *sh, uintptr_t key) ATTR_NONNULL(1);
-int     BLI_smallhash_count(SmallHash *sh)  ATTR_NONNULL(1);
+void   *BLI_smallhash_lookup(const SmallHash *sh, uintptr_t key) ATTR_NONNULL(1) ATTR_WARN_UNUSED_RESULT;
+void  **BLI_smallhash_lookup_p(const SmallHash *sh, uintptr_t key) ATTR_NONNULL(1) ATTR_WARN_UNUSED_RESULT;
+bool    BLI_smallhash_haskey(const SmallHash *sh, uintptr_t key) ATTR_NONNULL(1);
+int     BLI_smallhash_count(const SmallHash *sh)  ATTR_NONNULL(1);
 void   *BLI_smallhash_iternext(SmallHashIter *iter, uintptr_t *key)  ATTR_NONNULL(1) ATTR_WARN_UNUSED_RESULT;
 void  **BLI_smallhash_iternext_p(SmallHashIter *iter, uintptr_t *key)  ATTR_NONNULL(1) ATTR_WARN_UNUSED_RESULT;
-void   *BLI_smallhash_iternew(SmallHash *sh, SmallHashIter *iter, uintptr_t *key) ATTR_NONNULL(1) ATTR_WARN_UNUSED_RESULT;
-void  **BLI_smallhash_iternew_p(SmallHash *sh, SmallHashIter *iter, uintptr_t *key) ATTR_NONNULL(1) ATTR_WARN_UNUSED_RESULT;
+void   *BLI_smallhash_iternew(const SmallHash *sh, SmallHashIter *iter, uintptr_t *key) ATTR_NONNULL(1) ATTR_WARN_UNUSED_RESULT;
+void  **BLI_smallhash_iternew_p(const SmallHash *sh, SmallHashIter *iter, uintptr_t *key) ATTR_NONNULL(1) ATTR_WARN_UNUSED_RESULT;
 /* void BLI_smallhash_print(SmallHash *sh); */ /* UNUSED */
 
 #ifdef DEBUG
index 8dd72406250eda0e7d2c9420dc9ea6ef158f9550..82d57dad7537e6bb5fba7d03a62300d0ac2fe801 100644 (file)
@@ -24,7 +24,7 @@
  *  \ingroup bli
  *
  * An (edge -> pointer) chaining hash table.
- * Using unordered int-paits as keys.
+ * Using unordered int-pairs as keys.
  *
  * \note Based on 'BLI_ghash.c', which is a more generalized hash-table
  * make sure these stay in sync.
index bb91f48d39672663318aa25dc6f55bf98eec4fdf..ba336ab6c4b81550f54b0cc3eb1c8c36dd3b0c25 100644 (file)
@@ -118,7 +118,7 @@ BLI_INLINE void smallhash_buckets_reserve(SmallHash *sh, const unsigned int nent
        }
 }
 
-BLI_INLINE SmallHashEntry *smallhash_lookup(SmallHash *sh, const uintptr_t key)
+BLI_INLINE SmallHashEntry *smallhash_lookup(const SmallHash *sh, const uintptr_t key)
 {
        SmallHashEntry *e;
        unsigned int h = smallhash_key(key);
@@ -284,28 +284,28 @@ bool BLI_smallhash_remove(SmallHash *sh, uintptr_t key)
 }
 #endif
 
-void *BLI_smallhash_lookup(SmallHash *sh, uintptr_t key)
+void *BLI_smallhash_lookup(const SmallHash *sh, uintptr_t key)
 {
        SmallHashEntry *e = smallhash_lookup(sh, key);
 
        return e ? e->val : NULL;
 }
 
-void **BLI_smallhash_lookup_p(SmallHash *sh, uintptr_t key)
+void **BLI_smallhash_lookup_p(const SmallHash *sh, uintptr_t key)
 {
        SmallHashEntry *e = smallhash_lookup(sh, key);
 
        return e ? &e->val : NULL;
 }
 
-bool BLI_smallhash_haskey(SmallHash *sh, uintptr_t key)
+bool BLI_smallhash_haskey(const SmallHash *sh, uintptr_t key)
 {
        SmallHashEntry *e = smallhash_lookup(sh, key);
 
        return (e != NULL);
 }
 
-int BLI_smallhash_count(SmallHash *sh)
+int BLI_smallhash_count(const SmallHash *sh)
 {
        return (int)sh->nentries;
 }
@@ -341,7 +341,7 @@ void **BLI_smallhash_iternext_p(SmallHashIter *iter, uintptr_t *key)
        return e ? &e->val : NULL;
 }
 
-void *BLI_smallhash_iternew(SmallHash *sh, SmallHashIter *iter, uintptr_t *key)
+void *BLI_smallhash_iternew(const SmallHash *sh, SmallHashIter *iter, uintptr_t *key)
 {
        iter->sh = sh;
        iter->i = 0;
@@ -349,7 +349,7 @@ void *BLI_smallhash_iternew(SmallHash *sh, SmallHashIter *iter, uintptr_t *key)
        return BLI_smallhash_iternext(iter, key);
 }
 
-void **BLI_smallhash_iternew_p(SmallHash *sh, SmallHashIter *iter, uintptr_t *key)
+void **BLI_smallhash_iternew_p(const SmallHash *sh, SmallHashIter *iter, uintptr_t *key)
 {
        iter->sh = sh;
        iter->i = 0;
index 8445da8ecab0848d4694de5dc6e70f922745a17c..ec90217940c754415592e3bbc760093a069b4980 100644 (file)
@@ -3125,7 +3125,7 @@ static int render_border_exec(bContext *C, wmOperator *op)
        }
 
        rd = RE_engine_get_render_data(re);
-       if ((rd->mode & (R_BORDER|R_CROP)) == (R_BORDER|R_CROP)) {
+       if ((rd->mode & (R_BORDER | R_CROP)) == (R_BORDER | R_CROP)) {
                BKE_report(op->reports, RPT_INFO, "Can not set border from a cropped render");
                return OPERATOR_CANCELLED;
        }