diff options
| author | Campbell Barton <ideasman42@gmail.com> | 2015-04-07 03:53:58 +0300 |
|---|---|---|
| committer | Campbell Barton <ideasman42@gmail.com> | 2015-04-07 03:53:58 +0300 |
| commit | 5217d2bc0e29695bea74950ff05a9215a85ff703 (patch) | |
| tree | f9c6761111496e748e7161f1998c531957bcc47c /source/blender/bmesh/intern/bmesh_mesh_validate.c | |
| parent | 808de65d9124d54ce82bcb1a4d84a78b3be7288c (diff) | |
Use BKE_edgehash_ensure_p where possible
Diffstat (limited to 'source/blender/bmesh/intern/bmesh_mesh_validate.c')
| -rw-r--r-- | source/blender/bmesh/intern/bmesh_mesh_validate.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/source/blender/bmesh/intern/bmesh_mesh_validate.c b/source/blender/bmesh/intern/bmesh_mesh_validate.c index 3a7a4f85e99..494da3b889b 100644 --- a/source/blender/bmesh/intern/bmesh_mesh_validate.c +++ b/source/blender/bmesh/intern/bmesh_mesh_validate.c @@ -86,20 +86,19 @@ bool BM_mesh_validate(BMesh *bm) /* check edges */ BM_ITER_MESH_INDEX (e, &iter, bm, BM_EDGES_OF_MESH, i) { - BMEdge *e_other; + void **val_p; if (e->v1 == e->v2) { ERRMSG("edge %d: duplicate index: %d", i, BM_elem_index_get(e->v1)); } - /* build edgehash at the same time */ - e_other = BLI_edgehash_lookup(edge_hash, BM_elem_index_get(e->v1), BM_elem_index_get(e->v2)); - if (e_other) { + if (BLI_edgehash_ensure_p(edge_hash, BM_elem_index_get(e->v1), BM_elem_index_get(e->v2), &val_p)) { + BMEdge *e_other = *val_p; ERRMSG("edge %d, %d: are duplicates", i, BM_elem_index_get(e_other)); } else { - BLI_edgehash_insert(edge_hash, BM_elem_index_get(e->v1), BM_elem_index_get(e->v2), e); + *val_p = e; } } |