Skip to content

Revert "assert: change the do-while of assert to a conditional expression"#11740

Merged
xiaoxiang781216 merged 3 commits into
masterfrom
revert-11734-assert
Feb 22, 2024
Merged

Revert "assert: change the do-while of assert to a conditional expression"#11740
xiaoxiang781216 merged 3 commits into
masterfrom
revert-11734-assert

Conversation

@masayuki2009

Copy link
Copy Markdown
Contributor

Reverts #11734
since there are many build failures.

@yf13

yf13 commented Feb 22, 2024

Copy link
Copy Markdown
Contributor

@masayuki2009 I've sent #11739 in this repo and #2301 in apps repo, do you still want revert?

@masayuki2009

Copy link
Copy Markdown
Contributor Author

@masayuki2009 I've sent #11739 in this repo and #2301 in apps repo, do you still want revert?

@yf13
Thanks, but I think it's not enough.

@xiaoxiang781216

Copy link
Copy Markdown
Contributor

@masayuki2009 I've sent #11739 in this repo and #2301 in apps repo, do you still want revert?

@masayuki2009 let's fix the minor warning today instead reverting the changing?

@Gary-Hobson

Copy link
Copy Markdown
Contributor

@masayuki2009 I've sent #11739 in this repo and #2301 in apps repo, do you still want revert?

@masayuki2009 let's fix the minor warning today instead reverting the changing?

This bug is fixed in the following two PRs:
#11742
apache/nuttx-apps#2302

@xiaoxiang781216

xiaoxiang781216 commented Feb 22, 2024

Copy link
Copy Markdown
Contributor

Since DEBUGASSRT is nuttx specific macro, we can only change assert, so let's merge this patch first. @Gary-Hobson please provide a new patch which just change assert.

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.

5 participants