v01d commented on a change in pull request #2116:
URL: https://github.com/apache/incubator-nuttx/pull/2116#discussion_r511931050
##########
File path: arch/sim/Kconfig
##########
@@ -99,7 +99,7 @@ endchoice
config SIM_WALLTIME
bool "Run the simulation at a fixed cadence in near real-time"
- default n
+ default y
Review comment:
If the default is `y`, I think it is OK to have it depend on it. When a
default is `n`, the depend makes it hard to discover the option, in that case
`select` is more useful.
----------------------------------------------------------------
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:
[email protected]
