davids5 commented on pull request #2906:
URL: https://github.com/apache/incubator-nuttx/pull/2906#issuecomment-784991084


   > is it not enough with the comment before valloc?
   
   What comment? Did I miss it? Maybe add one, that labels it as `obsolete` and 
that will work. 
   
    > And it isn't good to mention LTP in a general function.
   
   LOL... I was referring to documenting it in the Kconfig. --help - Something 
to the effect "This is an obsolete function that can be enabled to be used by 
some third partly libraries that require it."  
   
   > Beginner also hard to do the right configuration
   
   Yes very true. You can say the same about a Beginner developer, new to 
NuttX, dealing with lots of functions that were added to to make the build work 
for a project that should not be used. and it not referenced anywhere in this 
project.
   
   It is a case of information overload. Unqualified information - leaves the 
user overwhelmed.  If you qualify it it it can be ignored.
   
   I do see your point. I agree with it.  Please add a comment and call it a 
done.
   


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to