minor improvements

- calc normals only check flag when needed.
- keymap, dont get name unless its needed.
- keymap, avoid property lookup.
- idprop debug print, include pointer, helpful for troubleshooting.

2 files changed, 5 insertions, 2 deletions

diff --git a/source/blender/bmesh/intern/bmesh_queries.c b/source/blender/bmesh/intern/bmesh_queries.c
index c0c4e0f0209..a00bef0228b 100644
--- a/source/blender/bmesh/intern/bmesh_queries.c
+++ b/source/blender/bmesh/intern/bmesh_queries.c
@@ -1761,7 +1761,7 @@ float BM_mesh_calc_volume(BMesh *bm, bool is_signed)
 
 /**
  * TODO (as we need)
- * - option to walk over edges.
+ * - option to walk over faces by verts.
  * - option to walk over non manifold edges.
  *
  * \param bm  the BMesh.
diff --git a/source/blender/bmesh/operators/bmo_normals.c b/source/blender/bmesh/operators/bmo_normals.c
index 7116ef82a32..c4200a37748 100644
--- a/source/blender/bmesh/operators/bmo_normals.c
+++ b/source/blender/bmesh/operators/bmo_normals.c
@@ -166,7 +166,10 @@ void bmo_recalc_face_normals_exec(BMesh *bm, BMOperator *op)
 
 		for (j = 0; j < fg_len; j++) {
 			faces_grp[j] = faces_arr[groups_array[fg_sta + j]];
-			is_calc |= BMO_elem_flag_test_bool(bm, faces_grp[j], FACE_FLAG);
+
+			if (is_calc == false) {
+				is_calc = BMO_elem_flag_test_bool(bm, faces_grp[j], FACE_FLAG);
+			}
 		}
 
 		if (is_calc) {