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 2020/04/30 16:10:22 UTC

[GitHub] [incubator-nuttx] patacongo edited a comment on issue #915: Change all function prefix from up_ to arch_ in include/nuttx/arch.h

patacongo edited a comment on issue #915:
URL: https://github.com/apache/incubator-nuttx/issues/915#issuecomment-621950081


   In #916, the error is in the name up_fullcontextrestore(tcb->xcp.regs); which is incorrect.  That should be arm_fullcontextrestore().  The naming in other ARM architectures is incorrect.  The other architectures are not following the naming standard.  ARMv8-M is correctly following the naming standard.
   
   So yes, there will be issues UNTIL other ARM architectures are mad to follow the naming standard.  Let's do that!!  Let's fix the other ARM architectures and not screw up the naming standard.
   


----------------------------------------------------------------
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.

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