From aad516127efb5c75e23ba0f89b7bdbb4cd43356e Mon Sep 17 00:00:00 2001 From: JosePizarro3 Date: Mon, 15 Apr 2024 14:35:20 +0200 Subject: [PATCH] Delete comment in normalize.md --- docs/normalize.md | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/docs/normalize.md b/docs/normalize.md index 500a9251..4a838407 100644 --- a/docs/normalize.md +++ b/docs/normalize.md @@ -46,6 +46,9 @@ Now, `MetainfoNormalizer` will be ran on the `ParentSection`. Applying **rule 1* ### Avoiding double execution of public functions -When the `MetainfoNormalize` is ran, and the `normalize()` functions are executed, these usually have calls for calling private and public functions in the base classes which might have been used already during parsing the files. However, **we have checks to ensure that this will not happen**. As an user, you can decide whether running then in your parsers for helping you during the process, or leave it up to the `normalize()` functions to decide if these are ran or not. +When the `MetainfoNormalize` is ran, and the `normalize()` functions are executed, these usually have calls for calling private and public functions in the base classes which might have been used already during parsing the files. However, we have checks to ensure that this will not happen. As an user, you can decide whether running then in your parsers for helping you during the process, or leave it up to the `normalize()` functions to decide if these are ran or not. -**JMP: I realized we have to make sure that a function has not been used already. Probably we can use the m_cache for this, but most importantly: does this make sense? I.e., do we want to overwrite running public functions when normalize() is ran or do we want to avoid this? Maybe someone used wrongly the function, or do we simply overlook this and ignore it?** + +