HedongGao opened a new pull request, #17652:
URL: https://github.com/apache/nuttx/pull/17652

   *Note: Please adhere to [Contributing 
Guidelines](https://github.com/apache/nuttx/blob/master/CONTRIBUTING.md).*
   
   ## Summary
   IEEE802154_STATUS_STRING is defined in the .h file, which will generate 
compilation warning information under some compilers.  Moved the position of 
the variable to fix this issue.  At the same time, changed the name of the 
variable to unify the code style.
   
   ## Impact
   
   This change has rename `static const char *IEEE802154_STATUS_STRING[]` to 
`FAR const char *g_ieee802154_status_string[]`.  It will affect 
wireless/ieee802154/i8sak/*.c in nuttx-apps repository which used this var. So, 
I submitted another patch to the nuttx apps repository:
   [apps 
pr](https://github.com/apache/nuttx-apps/compare/master...HedongGao:nuttx-apps:wireless?expand=1)
   
   ## Testing
   
   gcc compiler has no warning infor for 
`nuttx-apps/wireless/ieee802154/i8sak`, but tasking compiler has warning 
information.
   
   warnning log: unused variable "IEEE802154_STATUS_STRING"
   
   After the issue is fixed, the warning dispeared, and the results compiled 
with both compilers are normal.
   
   
   **_PRs without testing information will not be accepted. We will
   request test logs._**
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to