diff options
author | Campbell Barton <campbell@blender.org> | 2022-08-12 01:57:38 +0300 |
---|---|---|
committer | Campbell Barton <campbell@blender.org> | 2022-08-12 01:57:38 +0300 |
commit | 62d1ed0120b26cd0860aa430ba2a03da74968556 (patch) | |
tree | 9b7c75f71f5e7d31b1f26cadd96da900c8aeedb3 | |
parent | 344919240c8bd9ad2457f1775596bf0a1bd7560f (diff) |
Cleanup: replace term face with poly
Be consistent with naming to avoid mixing MPoly/MFace.
5 files changed, 34 insertions, 34 deletions
diff --git a/source/blender/blenkernel/intern/mesh_evaluate.cc b/source/blender/blenkernel/intern/mesh_evaluate.cc index ebd9798be7b..9dba8eab340 100644 --- a/source/blender/blenkernel/intern/mesh_evaluate.cc +++ b/source/blender/blenkernel/intern/mesh_evaluate.cc @@ -761,7 +761,7 @@ void BKE_mesh_flush_hidden_from_verts(Mesh *me) } hide_edge.finish(); - /* Hide faces when any of their vertices are hidden. */ + /* Hide polygons when any of their vertices are hidden. */ SpanAttributeWriter<bool> hide_poly = attributes.lookup_or_add_for_write_only_span<bool>( ".hide_poly", ATTR_DOMAIN_FACE); for (const int i : polys.index_range()) { @@ -787,7 +787,7 @@ void BKE_mesh_flush_hidden_from_polys(Mesh *me) attributes.remove(".hide_edge"); return; } - const VArraySpan<bool> hide_face_span{hide_poly}; + const VArraySpan<bool> hide_poly_span{hide_poly}; const Span<MPoly> polys(me->mpoly, me->totpoly); const Span<MLoop> loops(me->mloop, me->totloop); SpanAttributeWriter<bool> hide_vert = attributes.lookup_or_add_for_write_only_span<bool>( @@ -795,9 +795,9 @@ void BKE_mesh_flush_hidden_from_polys(Mesh *me) SpanAttributeWriter<bool> hide_edge = attributes.lookup_or_add_for_write_only_span<bool>( ".hide_edge", ATTR_DOMAIN_EDGE); - /* Hide all edges or vertices connected to hidden faces. */ + /* Hide all edges or vertices connected to hidden polygons. */ for (const int i : polys.index_range()) { - if (hide_face_span[i]) { + if (hide_poly_span[i]) { const MPoly &poly = polys[i]; for (const MLoop &loop : loops.slice(poly.loopstart, poly.totloop)) { hide_vert.span[loop.v] = true; @@ -805,9 +805,9 @@ void BKE_mesh_flush_hidden_from_polys(Mesh *me) } } } - /* Unhide vertices and edges connected to visible faces. */ + /* Unhide vertices and edges connected to visible polygons. */ for (const int i : polys.index_range()) { - if (!hide_face_span[i]) { + if (!hide_poly_span[i]) { const MPoly &poly = polys[i]; for (const MLoop &loop : loops.slice(poly.loopstart, poly.totloop)) { hide_vert.span[loop.v] = false; diff --git a/source/blender/blenkernel/intern/mesh_legacy_convert.cc b/source/blender/blenkernel/intern/mesh_legacy_convert.cc index 04c1850f4ae..975197a7d58 100644 --- a/source/blender/blenkernel/intern/mesh_legacy_convert.cc +++ b/source/blender/blenkernel/intern/mesh_legacy_convert.cc @@ -906,11 +906,11 @@ void BKE_mesh_legacy_convert_hide_layers_to_flags(Mesh *mesh) }); MutableSpan<MPoly> polygons(mesh->mpoly, mesh->totpoly); - const VArray<bool> hide_face = attributes.lookup_or_default<bool>( + const VArray<bool> hide_poly = attributes.lookup_or_default<bool>( ".hide_poly", ATTR_DOMAIN_FACE, false); threading::parallel_for(polygons.index_range(), 4096, [&](IndexRange range) { for (const int i : range) { - SET_FLAG_FROM_TEST(polygons[i].flag, hide_face[i], ME_HIDE); + SET_FLAG_FROM_TEST(polygons[i].flag, hide_poly[i], ME_HIDE); } }); } @@ -952,14 +952,14 @@ void BKE_mesh_legacy_convert_flags_to_hide_layers(Mesh *mesh) if (std::any_of(polygons.begin(), polygons.end(), [](const MPoly &poly) { return poly.flag & ME_HIDE; })) { - SpanAttributeWriter<bool> hide_face = attributes.lookup_or_add_for_write_only_span<bool>( + SpanAttributeWriter<bool> hide_poly = attributes.lookup_or_add_for_write_only_span<bool>( ".hide_poly", ATTR_DOMAIN_FACE); threading::parallel_for(polygons.index_range(), 4096, [&](IndexRange range) { for (const int i : range) { - hide_face.span[i] = polygons[i].flag & ME_HIDE; + hide_poly.span[i] = polygons[i].flag & ME_HIDE; } }); - hide_face.finish(); + hide_poly.finish(); } } diff --git a/source/blender/bmesh/intern/bmesh_mesh_convert.cc b/source/blender/bmesh/intern/bmesh_mesh_convert.cc index cdaaabd1f05..b9c004b5392 100644 --- a/source/blender/bmesh/intern/bmesh_mesh_convert.cc +++ b/source/blender/bmesh/intern/bmesh_mesh_convert.cc @@ -950,7 +950,7 @@ static void write_elem_flag_to_attribute(blender::bke::MutableAttributeAccessor static void convert_bmesh_hide_flags_to_mesh_attributes(BMesh &bm, const bool need_hide_vert, const bool need_hide_edge, - const bool need_hide_face, + const bool need_hide_poly, Mesh &mesh) { using namespace blender; @@ -959,7 +959,7 @@ static void convert_bmesh_hide_flags_to_mesh_attributes(BMesh &bm, BLI_assert(CustomData_get_layer_named(&bm.edata, CD_PROP_BOOL, ".hide_edge") == nullptr); BLI_assert(CustomData_get_layer_named(&bm.pdata, CD_PROP_BOOL, ".hide_poly") == nullptr); - if (!(need_hide_vert || need_hide_edge || need_hide_face)) { + if (!(need_hide_vert || need_hide_edge || need_hide_poly)) { return; } @@ -975,7 +975,7 @@ static void convert_bmesh_hide_flags_to_mesh_attributes(BMesh &bm, return BM_elem_flag_test(BM_edge_at_index(&bm, i), BM_ELEM_HIDDEN); }); write_elem_flag_to_attribute( - attributes, ".hide_poly", ATTR_DOMAIN_FACE, need_hide_face, [&](const int i) { + attributes, ".hide_poly", ATTR_DOMAIN_FACE, need_hide_poly, [&](const int i) { return BM_elem_flag_test(BM_face_at_index(&bm, i), BM_ELEM_HIDDEN); }); } @@ -1038,7 +1038,7 @@ void BM_mesh_bm_to_me(Main *bmain, BMesh *bm, Mesh *me, const struct BMeshToMesh bool need_hide_vert = false; bool need_hide_edge = false; - bool need_hide_face = false; + bool need_hide_poly = false; /* Clear normals on the mesh completely, since the original vertex and polygon count might be * different than the BMesh's. */ @@ -1114,7 +1114,7 @@ void BM_mesh_bm_to_me(Main *bmain, BMesh *bm, Mesh *me, const struct BMeshToMesh mpoly->mat_nr = f->mat_nr; mpoly->flag = BM_face_flag_to_mflag(f); if (BM_elem_flag_test(f, BM_ELEM_HIDDEN)) { - need_hide_face = true; + need_hide_poly = true; } l_iter = l_first = BM_FACE_FIRST_LOOP(f); @@ -1209,7 +1209,7 @@ void BM_mesh_bm_to_me(Main *bmain, BMesh *bm, Mesh *me, const struct BMeshToMesh } convert_bmesh_hide_flags_to_mesh_attributes( - *bm, need_hide_vert, need_hide_edge, need_hide_face, *me); + *bm, need_hide_vert, need_hide_edge, need_hide_poly, *me); BKE_mesh_update_customdata_pointers(me, false); @@ -1306,7 +1306,7 @@ void BM_mesh_bm_to_me_for_eval(BMesh *bm, Mesh *me, const CustomData_MeshMasks * bool need_hide_vert = false; bool need_hide_edge = false; - bool need_hide_face = false; + bool need_hide_poly = false; /* Clear normals on the mesh completely, since the original vertex and polygon count might be * different than the BMesh's. */ @@ -1381,7 +1381,7 @@ void BM_mesh_bm_to_me_for_eval(BMesh *bm, Mesh *me, const CustomData_MeshMasks * mp->totloop = efa->len; mp->flag = BM_face_flag_to_mflag(efa); if (BM_elem_flag_test(efa, BM_ELEM_HIDDEN)) { - need_hide_face = true; + need_hide_poly = true; } mp->loopstart = j; @@ -1404,7 +1404,7 @@ void BM_mesh_bm_to_me_for_eval(BMesh *bm, Mesh *me, const CustomData_MeshMasks * bm->elem_index_dirty &= ~(BM_FACE | BM_LOOP); convert_bmesh_hide_flags_to_mesh_attributes( - *bm, need_hide_vert, need_hide_edge, need_hide_face, *me); + *bm, need_hide_vert, need_hide_edge, need_hide_poly, *me); me->cd_flag = BM_mesh_cd_flag_from_bmesh(bm); } diff --git a/source/blender/draw/intern/mesh_extractors/extract_mesh_vbo_pos_nor.cc b/source/blender/draw/intern/mesh_extractors/extract_mesh_vbo_pos_nor.cc index 4d2a10e83ce..313838be9e8 100644 --- a/source/blender/draw/intern/mesh_extractors/extract_mesh_vbo_pos_nor.cc +++ b/source/blender/draw/intern/mesh_extractors/extract_mesh_vbo_pos_nor.cc @@ -87,7 +87,7 @@ static void extract_pos_nor_iter_poly_mesh(const MeshRenderData *mr, void *_data) { MeshExtract_PosNor_Data *data = static_cast<MeshExtract_PosNor_Data *>(_data); - const bool face_hidden = mr->hide_poly && mr->hide_poly[mp_index]; + const bool poly_hidden = mr->hide_poly && mr->hide_poly[mp_index]; const MLoop *mloop = mr->mloop; const int ml_index_end = mp->loopstart + mp->totloop; @@ -100,7 +100,7 @@ static void extract_pos_nor_iter_poly_mesh(const MeshRenderData *mr, copy_v3_v3(vert->pos, mv->co); vert->nor = data->normals[ml->v].low; /* Flag for paint mode overlay. */ - if (face_hidden || vert_hidden || + if (poly_hidden || vert_hidden || ((mr->extract_type == MR_EXTRACT_MAPPED) && (mr->v_origindex) && (mr->v_origindex[ml->v] == ORIGINDEX_NONE))) { vert->nor.w = -1; @@ -434,7 +434,7 @@ static void extract_pos_nor_hq_iter_poly_mesh(const MeshRenderData *mr, void *_data) { MeshExtract_PosNorHQ_Data *data = static_cast<MeshExtract_PosNorHQ_Data *>(_data); - const bool face_hidden = mr->hide_poly && mr->hide_poly[mp - mr->mpoly]; + const bool poly_hidden = mr->hide_poly && mr->hide_poly[mp - mr->mpoly]; const MLoop *mloop = mr->mloop; const int ml_index_end = mp->loopstart + mp->totloop; @@ -448,7 +448,7 @@ static void extract_pos_nor_hq_iter_poly_mesh(const MeshRenderData *mr, copy_v3_v3_short(vert->nor, data->normals[ml->v].high); /* Flag for paint mode overlay. */ - if (face_hidden || vert_hidden || + if (poly_hidden || vert_hidden || ((mr->extract_type == MR_EXTRACT_MAPPED) && (mr->v_origindex) && (mr->v_origindex[ml->v] == ORIGINDEX_NONE))) { vert->nor[3] = -1; diff --git a/source/blender/editors/mesh/editface.cc b/source/blender/editors/mesh/editface.cc index 96a493cba49..b1e1e94e5f7 100644 --- a/source/blender/editors/mesh/editface.cc +++ b/source/blender/editors/mesh/editface.cc @@ -82,12 +82,12 @@ void paintface_flush_flags(bContext *C, me_orig->mpoly[i].flag = me->mpoly[i].flag; } if (flush_hidden) { - const VArray<bool> hide_face_me = attributes_me.lookup_or_default<bool>( + const VArray<bool> hide_poly_me = attributes_me.lookup_or_default<bool>( ".hide_poly", ATTR_DOMAIN_FACE, false); - bke::SpanAttributeWriter<bool> hide_face_orig = + bke::SpanAttributeWriter<bool> hide_poly_orig = attributes_orig.lookup_or_add_for_write_only_span<bool>(".hide_poly", ATTR_DOMAIN_FACE); - hide_face_me.materialize(hide_face_orig.span); - hide_face_orig.finish(); + hide_poly_me.materialize(hide_poly_orig.span); + hide_poly_orig.finish(); } /* Mesh polys => Final derived polys */ @@ -103,17 +103,17 @@ void paintface_flush_flags(bContext *C, polys[i].flag = mp_orig->flag; } } - const VArray<bool> hide_face_orig = attributes_orig.lookup_or_default<bool>( + const VArray<bool> hide_poly_orig = attributes_orig.lookup_or_default<bool>( ".hide_poly", ATTR_DOMAIN_FACE, false); - bke::SpanAttributeWriter<bool> hide_face_eval = + bke::SpanAttributeWriter<bool> hide_poly_eval = attributes_eval.lookup_or_add_for_write_only_span<bool>(".hide_poly", ATTR_DOMAIN_FACE); for (const int i : IndexRange(me_eval->totpoly)) { - const int orig_face_index = index_array[i]; - if (orig_face_index != ORIGINDEX_NONE) { - hide_face_eval.span[i] = hide_face_orig[orig_face_index]; + const int orig_poly_index = index_array[i]; + if (orig_poly_index != ORIGINDEX_NONE) { + hide_poly_eval.span[i] = hide_poly_orig[orig_poly_index]; } } - hide_face_eval.finish(); + hide_poly_eval.finish(); updated = true; } |