You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2022/07/13 17:39:07 UTC

[GitHub] [incubator-nuttx] gustavonihei opened a new pull request, #6609: Bugfix/debugpanic

gustavonihei opened a new pull request, #6609:
URL: https://github.com/apache/incubator-nuttx/pull/6609

   ## Summary
   This PR intends to provide a minor improvement to code readability by converting some `DEBUGASSERT(false)` calls into `DEBUGPANIC()`.
   
   ## Impact
   Both macros are similar, should have no impact to the generated code.
   
   ## Testing
   CI build pass.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-nuttx] gustavonihei commented on pull request #6609: Convert DEBUGASSERT(false) into more intuitive DEBUGPANIC()

Posted by GitBox <gi...@apache.org>.
gustavonihei commented on PR #6609:
URL: https://github.com/apache/incubator-nuttx/pull/6609#issuecomment-1183664691

   > Please convert `DEBUGASSERT(0);` as well
   
   Just pushed the requested changes.
   Thanks for the suggestion!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6609: Convert DEBUGASSERT(false) into more intuitive DEBUGPANIC()

Posted by GitBox <gi...@apache.org>.
pkarashchenko commented on PR #6609:
URL: https://github.com/apache/incubator-nuttx/pull/6609#issuecomment-1183619186

   Please convert `DEBUGASSERT(0);` as well


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #6609: Convert DEBUGASSERT(false) into more intuitive DEBUGPANIC()

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 merged PR #6609:
URL: https://github.com/apache/incubator-nuttx/pull/6609


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org