Attention is currently required from: Hoernchen, osmith.
Jenkins Builder has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42330?usp=email )
Change subject: fw source: force reformat once and for all ......................................................................
Patch Set 3:
(3 comments)
File sysmoOCTSIM/dfu_descriptors.h:
Robot Comment from checkpatch (run ID ): https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42330/comment/6bfdb332_fde... : PS3, Line 30: #define DFURT_IF_DESCRIPTOR_STRUCT \ macros should not use a trailing semicolon
File sysmoOCTSIM/talloc.h:
Robot Comment from checkpatch (run ID ): https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42330/comment/1997cc8b_eb8... : PS3, Line 880: void *talloc_pooled_object(const void *ctx, #type, unsigned num_subobjects, size_t total_subobjects_size); function definition argument 'type' should also have an identifier name
File sysmoOCTSIM/talloc.c:
Robot Comment from checkpatch (run ID ): https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42330/comment/80be2045_f5e... : PS3, Line 251: #define TC_UNDEFINE_GROW_CHUNK(_tc, _new_size) \ Single statement macros should not use a do {} while (0) loop