Skip to content

Comments

Clang format#6

Draft
pillo79 wants to merge 3 commits intomainfrom
clang-format
Draft

Clang format#6
pillo79 wants to merge 3 commits intomainfrom
clang-format

Conversation

@pillo79
Copy link
Owner

@pillo79 pillo79 commented Feb 17, 2026

No description provided.

@pillo79 pillo79 force-pushed the clang-format branch 26 times, most recently from 82f7059 to bea7b12 Compare February 18, 2026 13:54
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clang-format issues found\n

Formatting issues found in cores/arduino/zephyrCommon.cpp\n


https://github.com/pillo79/ArduinoCore-zephyr/blob//cores/arduino/zephyrCommon.cpp#L56-L56

	is_first_appearance(0, i, ((size_t) - 1), DEVICE_DT_GET(DT_GPIO_CTLR_BY_IDX(n, p, i)),         \

Clang-format issues found\n

Formatting issues found in loader/llext_exports.c\n


https://github.com/pillo79/ArduinoCore-zephyr/blob//loader/llext_exports.c#L11-L12

#define FORCE_EXPORT_SYM(name)                                                                     \
	extern void name(void);                                                                        \

https://github.com/pillo79/ArduinoCore-zephyr/blob//loader/llext_exports.c#L255-L256

// FORCE_EXPORT_SYM(k_timer_user_data_set);
// FORCE_EXPORT_SYM(k_timer_start);

https://github.com/pillo79/ArduinoCore-zephyr/blob//loader/llext_exports.c#L307-L307

#if defined(CONFIG_CPP)

Clang-format issues found\n

Formatting issues found in loader/main.c\n


https://github.com/pillo79/ArduinoCore-zephyr/blob//loader/main.c#L145-L145

	void matrixPlay(uint8_t * buf, uint32_t len);

https://github.com/pillo79/ArduinoCore-zephyr/blob//loader/main.c#L147-L147

	void matrixGrayscaleWrite(uint8_t * buf);

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Clang-format issues found\n

Formatting issues found in cores/arduino/zephyrCommon.cpp\n


is_first_appearance(0, i, ((size_t)-1), DEVICE_DT_GET(DT_GPIO_CTLR_BY_IDX(n, p, i)), \

	is_first_appearance(0, i, ((size_t) - 1), DEVICE_DT_GET(DT_GPIO_CTLR_BY_IDX(n, p, i)),         \

Clang-format issues found\n

Formatting issues found in loader/llext_exports.c\n


#define FORCE_EXPORT_SYM(name) \
extern void name(void); \

#define FORCE_EXPORT_SYM(name)                                                                     \
	extern void name(void);                                                                        \

//FORCE_EXPORT_SYM(k_timer_user_data_set);
//FORCE_EXPORT_SYM(k_timer_start);

// FORCE_EXPORT_SYM(k_timer_user_data_set);
// FORCE_EXPORT_SYM(k_timer_start);

#if defined (CONFIG_CPP)

#if defined(CONFIG_CPP)

Clang-format issues found\n

Formatting issues found in loader/main.c\n


void matrixPlay(uint8_t *buf, uint32_t len);

	void matrixPlay(uint8_t * buf, uint32_t len);

void matrixGrayscaleWrite(uint8_t *buf);

	void matrixGrayscaleWrite(uint8_t * buf);

@pillo79 pillo79 force-pushed the clang-format branch 2 times, most recently from cef3432 to e87e4cf Compare February 18, 2026 14:14
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please address the formatting issues highlighted in the review comments.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please address the formatting issues highlighted in the review comments.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please address the formatting issues highlighted in the review comments.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please address the formatting issues highlighted in the review comments.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please address the formatting issues highlighted in the review comments.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please address the formatting issues highlighted in the review comments.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Formatting issues have been resolved, thanks!

@pillo79 pillo79 force-pushed the clang-format branch 8 times, most recently from 1d80cbb to 7c4f96b Compare February 18, 2026 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant