juniskane opened a new pull request #909:
URL: https://github.com/apache/incubator-nuttx-apps/pull/909


   ## Summary
   
   Macro NULL is defined in multiple standard library headers, including 
<stdio.h> which is included by ostest source files, on all POSIX-conformant 
systems. Since the purpose of testing suites is to find errors, it makes no 
sense for them to try to "fix" broken headers missing normal functionality. 
This PR removes such kludges. Much better if test suite fails compilation if 
there is a problem with libc header files.
   
   Also C++ defines NULL as 0, not (void *)0.
   
   ## Impact
   N/A
   
   ## Testing
   ostest still passes on my systems.
   


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to