gustavonihei commented on pull request #3475:
URL: https://github.com/apache/incubator-nuttx/pull/3475#issuecomment-815577782


   > This kind of short, unadorned naming will always lead to problems and name 
collisions. Again, I recommend that you consider CONFIG_INPUT_TOUCHSCREEN and 
CONFIG_INPUT_MOUSE instead of CONFIG_TOUCHSCREEN and CONFIG_MOUSE. That is must 
less likely to conflict.
   
   Sorry, I may have not made myself clear. I agree with using 
`CONFIG_INPUT_TOUCHSCREEN` as well as `CONFIG_INPUT_MOUSE`, which would fit to 
what I described as `CONFIG_<SUBSYSTEM>_*`. It is just that I was intending to 
this renaming in a later PR.
   For now, since TOUCHSCREEN is being created, although it won't follow the 
style from the other configs, it may be created in the most suitable template. 
I'll change it to `CONFIG_INPUT_TOUCHSCREEN`.


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


Reply via email to