maskit commented on code in PR #10785:
URL: https://github.com/apache/trafficserver/pull/10785#discussion_r1396444512


##########
include/tscore/ink_platform.h:
##########
@@ -60,7 +60,7 @@
 struct ifafilt;
 #include <net/if.h>
 
-#ifdef HAVE_STDLIB_H
+#if __has_include(<stdlib.h>)

Review Comment:
   Yeah, I agree with @dragon512 on this one . I can't think of any system that 
doesn't need stdlib.h. But at the same time, this is to remove an unnecessary 
check in CMakeLists.txt. If there is any doubt I'm ok with this change.



-- 
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: github-unsubscr...@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to