Fix parameter redifinition

Parameters and layer files are included twice, in forward.hpp and forward.cpp.
However these generated files were not protected for multi-inclusion, leading to pointer and variable redefinition.

changed milestone to %aidge v0.8.0

added Fix 🔥🔥 StatusReview Ready TopicExport labels

requested review from @cmoineau

assigned to @axelfarr

mentioned in issue #43 (closed)

mentioned in issue #44 (closed)

Should we include this fix in aidge v0.7.0 ?

Yes, otherwise the export will be broken in 0.7.0!

However, I think the fix should be to add guards in the included files, or not include them in both forward.cpp and forward.h. The #pragma once directive is not standard and should be avoided.

aidge_core!476 (merged) Does this :)

@cmoineau So this MR is not necessary? Can we close it?

Need to see with @axelfarr

Actually, I kept a small aritfact issue so my mr didn't fully fix, I will push something in aidge core dev and rerun export cpp pipeline.

https://gitlab.eclipse.org/eclipse/aidge/aidge_export_cpp/-/pipelines/80317 fixed the pipeline sorry for the oversight yesterday...

changed milestone to %aidge v0.7.0

added Important ❗ label

Closing, this is already fixed.

closed