XinStellaris commented on code in PR #7808:
URL: https://github.com/apache/nuttx/pull/7808#discussion_r1045339088


##########
arch/avr/src/common/avr_assert.c:
##########
@@ -23,102 +23,12 @@
  ****************************************************************************/
 
 #include <nuttx/config.h>
-
-#include <stdint.h>
-#include <stdlib.h>
-#include <assert.h>
-#include <debug.h>

Review Comment:
   This is because some arch(such as arm64_assert.c) put dump logic in 
xxx_assert.c, while some (such as mips_assert.c) put dump logic in another 
xxxx_dumpstate.c.
   Unless we move all dump state logic into a common file, include files cannot 
be removed.
   As I have mentioned before, moving all dump state logic together will be in 
another patch (Lots of work). So how about we focus on common logic other than 
dump state in this patch?



-- 
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-unsubscr...@nuttx.apache.org

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

Reply via email to