diff options
author | Paul Elder <paul.elder@ideasonboard.com> | 2021-04-22 16:47:03 +0900 |
---|---|---|
committer | Paul Elder <paul.elder@ideasonboard.com> | 2021-04-27 17:34:55 +0900 |
commit | 17e8f6f71c0a03bd011675f83e93912317527885 (patch) | |
tree | 495bf05e246827f05e51eb88f82de2bc947e0b86 /include/libcamera/ipa/ipa_interface.h | |
parent | 446cb01d14470298f881c7bed4100c073f3ef835 (diff) |
utils: ipc: Include instead of forward-declare CameraSensorInfo
For structs defined in core.mojom that have the skipHeader tag, if
they're only used in function parameters (in a mojom file) then a
forward-declaration is sufficient. However, if the struct is used in
another struct in a mojom file, then the forward-declaration is
insufficient, and the definition needs to be included. Do so for
CameraSensorInfo, which is the only forward-declared struct in
ipa_interface.h, and update the documentation comment.
Signed-off-by: Paul Elder <paul.elder@ideasonboard.com>
Tested-by: Umang Jain <umang.jain@ideasonboard.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Diffstat (limited to 'include/libcamera/ipa/ipa_interface.h')
-rw-r--r-- | include/libcamera/ipa/ipa_interface.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/include/libcamera/ipa/ipa_interface.h b/include/libcamera/ipa/ipa_interface.h index 5d99e2cf..dfe1b40a 100644 --- a/include/libcamera/ipa/ipa_interface.h +++ b/include/libcamera/ipa/ipa_interface.h @@ -18,15 +18,15 @@ #include <libcamera/geometry.h> #include <libcamera/signal.h> +#include "libcamera/internal/camera_sensor.h" + namespace libcamera { /* * Structs that are defined in core.mojom and have the skipHeader tag must be - * forward-declared or #included here. + * #included here. */ -struct CameraSensorInfo; - class IPAInterface { public: |