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/05/04 19:07:57 UTC

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #969: Rename clock_systimer to clock_systimetick

xiaoxiang781216 commented on pull request #969:
URL: https://github.com/apache/incubator-nuttx/pull/969#issuecomment-623648824


   > This change is too arbritrary. I don't think we should permit arbitrary changes with no justification.
   
   We have two similar functions:
   1.clock_systimespec return timespec(second plus nanosecond)
   2.clock_systimer return clock tick
   So I think clock_systimetick is a better name becuase:
   1.Align with clock_systimespec
   2.Reader can know the time unit immediately
   3.All other related timing function use 'time', it's very strange that we use 'timer' here.


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