[GitHub] [incubator-nuttx] patacongo edited a comment on issue #353: multiple definition of `up_mdelay'

[GitBox <gi...@apache.org>]

patacongo edited a comment on issue #353: multiple definition of `up_mdelay'
URL: https://github.com/apache/incubator-nuttx/issues/353#issuecomment-589959094
 
 
   Actually, it is drivers/timer/arch_timer.c and arch_rtc.c that ruined the modularity be using up_ prefix.  But architectural thinking to begin with.  But let's not make things worse.  We keep all logic in "architectural silos" and we must prevent dependencies from spread across silos.  That is forbidden by the INVIOLABLES.txt and the basic architectural principles of the OS.  Modularity is the most important attribute of a good system and strictly enforcing modularity, not matter how painful, is the ONLY way to keep the architecture from degenerating into spaghetti code.

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


With regards,
Apache Git Services