DOXYGEN Precompiler Wrappers in SPI Master #1095
thestumbler
started this conversation in
General
Replies: 1 comment
-
Mostly yes. We would need to rewrite this to use concepts, as that would actually check the interface properly. But we didn't have those back when we started these interfaces. |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
I was following some code while debugging SPI interactions, and ran into something that looked wrong at first glance. For example, in the file
src/modm/architecture/interface/spi_master.hpp
, a huge swath of function header declarations are wrapped with inside an#ifdef __DOXYGEN__
block. Why can all these function declarations be ignored if you simply don't want to generate documentation?Clearly this works and is intentional, as I see this kind of pattern in many different modules, some with
#ifdef
and some with#ifndef
. These SPI functions all get defined in the appropriate platform driver files anyway, so is the architecture layer there just for documentation?Beta Was this translation helpful? Give feedback.
All reactions