From 1d12d90a0fd6c3ae82d5cde7c40c9458a52d44cc Mon Sep 17 00:00:00 2001 From: Corentin Prigent Date: Mon, 21 Jul 2025 15:57:01 +0200 Subject: [PATCH 1/2] factorize loop over triangle vertices --- src/mmg3d/analys_3d.c | 37 +++++++++---------------------------- 1 file changed, 9 insertions(+), 28 deletions(-) diff --git a/src/mmg3d/analys_3d.c b/src/mmg3d/analys_3d.c index 7f3c94db0..100bb27c8 100644 --- a/src/mmg3d/analys_3d.c +++ b/src/mmg3d/analys_3d.c @@ -52,36 +52,17 @@ void MMG3D_set_reqBoundaries(MMG5_pMesh mesh) { for (k=1; k<=mesh->nt; k++) { ptt = &mesh->tria[k]; - if ( mesh->info.nosurf && (!(ptt->tag[0] & MG_REQ)) ) { - ptt->tag[0] |= MG_REQ; - ptt->tag[0] |= MG_NOSURF; - } - - if ( ptt->tag[0] & MG_PARBDY ) { - ptt->tag[0] |= MG_NOSURF; - ptt->tag[0] |= MG_REQ; - } - - if ( mesh->info.nosurf && (!(ptt->tag[1] & MG_REQ)) ) { - ptt->tag[1] |= MG_REQ; - ptt->tag[1] |= MG_NOSURF; - } - - if ( ptt->tag[1] & MG_PARBDY ) { - ptt->tag[1] |= MG_NOSURF; - ptt->tag[1] |= MG_REQ; - } - - if ( mesh->info.nosurf && (!(ptt->tag[2] & MG_REQ)) ) { - ptt->tag[2] |= MG_REQ; - ptt->tag[2] |= MG_NOSURF; - } + for (int i=0;i<3;i++) { + if ( mesh->info.nosurf && (!(ptt->tag[i] & MG_REQ)) ) { + ptt->tag[i] |= MG_REQ; + ptt->tag[i] |= MG_NOSURF; + } - if ( ptt->tag[2] & MG_PARBDY ) { - ptt->tag[2] |= MG_NOSURF; - ptt->tag[2] |= MG_REQ; + if ( ptt->tag[i] & MG_PARBDY ) { + ptt->tag[i] |= MG_NOSURF; + ptt->tag[i] |= MG_REQ; + } } - } return; } From c1ea635f5109733341b827f26a9bdca1585d0496 Mon Sep 17 00:00:00 2001 From: Corentin Prigent Date: Mon, 21 Jul 2025 17:06:28 +0200 Subject: [PATCH 2/2] typo --- src/mmg3d/analys_3d.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/mmg3d/analys_3d.c b/src/mmg3d/analys_3d.c index 100bb27c8..e443ca01c 100644 --- a/src/mmg3d/analys_3d.c +++ b/src/mmg3d/analys_3d.c @@ -63,6 +63,7 @@ void MMG3D_set_reqBoundaries(MMG5_pMesh mesh) { ptt->tag[i] |= MG_REQ; } } + } return; }