[Live-devel] GroupsockHelper for Android

[Chase, Brian P [US] (MS)]

Good day,

In the GroupsockHelper.cpp file there are couple of preprocessor conditional blocks that populate a struct differently depending on whether the platform is Android or anything else. With the latest version of Android's Native Development Kit (NDK), version 17, the headers included define that struct the same way as the other platforms, thus breaking the code in those preprocessor blocks.

Therefore, I propose to that the "#ifdef __ANDROID__" lines be replace with "#if ANDROID_NDK_OLD" and such as the following block added amidst the other '#include's:

#ifdef __ANDROID_NDK__
#include <android/ndk-version.h>
#define ANDROID_OLD_NDK __NDK_MAJOR__ < 17
#endif

I hope these changes will be useful. I have attached a patch file and resulting source file for clarity. Have a blessed day.

Regards,
-Brian P. Chase


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.live555.com/pipermail/live-devel/attachments/20181010/a1557db8/attachment-0001.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: GroupsockHelper_cpp.patch
Type: application/octet-stream
Size: 3144 bytes
Desc: GroupsockHelper_cpp.patch
URL: <http://lists.live555.com/pipermail/live-devel/attachments/20181010/a1557db8/attachment-0001.obj>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: GroupsockHelper.cpp
URL: <http://lists.live555.com/pipermail/live-devel/attachments/20181010/a1557db8/attachment-0001.ksh>