summaryrefslogtreecommitdiff
path: root/include/libcamera/ipa/core.mojom
diff options
context:
space:
mode:
authorPaul Elder <paul.elder@ideasonboard.com>2021-04-22 16:47:03 +0900
committerPaul Elder <paul.elder@ideasonboard.com>2021-04-27 17:34:55 +0900
commit17e8f6f71c0a03bd011675f83e93912317527885 (patch)
tree495bf05e246827f05e51eb88f82de2bc947e0b86 /include/libcamera/ipa/core.mojom
parent446cb01d14470298f881c7bed4100c073f3ef835 (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/core.mojom')
-rw-r--r--include/libcamera/ipa/core.mojom3
1 files changed, 1 insertions, 2 deletions
diff --git a/include/libcamera/ipa/core.mojom b/include/libcamera/ipa/core.mojom
index 5363f1c5..70de71ea 100644
--- a/include/libcamera/ipa/core.mojom
+++ b/include/libcamera/ipa/core.mojom
@@ -38,8 +38,7 @@
* implemented in ipa_data_serializer.h, as it cannot be defined in mojom
* - [skipHeader] and [skipSerdes] only work here in core.mojom.
* - If a struct definition has [skipHeader], then the header where the
- * struct is defined must be #included (or the struct forward-declared) in
- * ipa_interface.h
+ * struct is defined must be #included in ipa_interface.h
* - If a field in a struct has a FileDescriptor, but is not explicitly
* defined so in mojom, then the field must be marked with the [hasFd]
* attribute.