aseba icon indicating copy to clipboard operation
aseba copied to clipboard

Move doxygen comments of public declarations from cpp to header files

Open ypiguet-epfl opened this issue 6 years ago • 3 comments

At some places such as in aseba/compiler/compiler.cpp for method compile, the doxygen documentation of public methods is located in the cpp file. It would be better to put it in the header file (aseba/compiler/compiler.h), because the cpp file could be provided as compiled code and not readily available. That's already the case for some of the public declarations (e.g. struct SubroutineDescriptor).

ypiguet-epfl avatar Mar 06 '18 10:03 ypiguet-epfl

Would you be willing to make a PR ?

cor3ntin avatar Mar 06 '18 11:03 cor3ntin

I think the git/github work wouldn't be worth the trouble for someone like me whose PR are limited to minor code polish. This can wait.

ypiguet-epfl avatar Mar 06 '18 12:03 ypiguet-epfl

For information, this was a design choice: at the time the compiler was written, code folding wasn't very widely available in editors and so the idea was that either someone knows more or less the code and wants to get a quick look at the available functions, and thus reads the h file, or this person wants the detailed information, and reads the generated documentation. This rationale might still be valid for people using editors such as vim.

stephanemagnenat avatar Mar 06 '18 16:03 stephanemagnenat