Please revert changeset 2c54272792e25f09d0f21be40f8d1d9204279495

Although SDL_android.h is not intended to be included by client code
sometimes it needed. For example you need JNIEnv pointer to make JNI calls
to modified SDLActivity.java (video playback, facebook integration,
in-apps). It seems a bit weird to write:

extern “C” {
#include “SDL_android.h”
}

in my AndroidJNI.cpp file.

Regards, Alexey.

Done!On Sun, Aug 4, 2013 at 5:23 AM, Alexey Petruchik wrote:

Although SDL_android.h is not intended to be included by client code
sometimes it needed. For example you need JNIEnv pointer to make JNI calls
to modified SDLActivity.java (video playback, facebook integration,
in-apps). It seems a bit weird to write:

extern “C” {
#include “SDL_android.h”
}

in my AndroidJNI.cpp file.

Regards, Alexey.


SDL mailing list
SDL at lists.libsdl.org
http://lists.libsdl.org/listinfo.cgi/sdl-libsdl.org

For the original problem reported here, I believe that the function
SDL_AndroidGetJNIEnv can be used instead. More such functions can be
function in SDL_system.h, along with a couple of helper functions for iOS.

More generally, yeah, there may be occasions where ones want to do some
things in a more direct manner. It is still safer to assume that
everything not coming from the ABI may be modified in a way that breaks
existing code any day, though.On 08/04/2013 07:24 PM, Sam Lantinga wrote:

Done!

On Sun, Aug 4, 2013 at 5:23 AM, Alexey Petruchik <i100500 at gmail.com <mailto:i100500 at gmail.com>> wrote:

Although SDL_android.h is not intended to be included by client code
sometimes it needed. For example you need JNIEnv pointer to make JNI
calls to modified SDLActivity.java (video playback, facebook
integration, in-apps). It seems a bit weird to write:

extern "C" {
#include "SDL_android.h"
}

in my AndroidJNI.cpp file.

Regards, Alexey.

_______________________________________________
SDL mailing list
SDL at lists.libsdl.org <mailto:SDL at lists.libsdl.org>
http://lists.libsdl.org/listinfo.cgi/sdl-libsdl.org

SDL mailing list
SDL at lists.libsdl.org
http://lists.libsdl.org/listinfo.cgi/sdl-libsdl.org

Thanks. I didn’t know about SDL_AndroidGetJNIEnv(). Shame on me)On Mon, Aug 5, 2013 at 10:08 PM, NY00123 wrote:

For the original problem reported here, I believe that the function
SDL_AndroidGetJNIEnv can be used instead. More such functions can be
function in SDL_system.h, along with a couple of helper functions for iOS.

More generally, yeah, there may be occasions where ones want to do some
things in a more direct manner. It is still safer to assume that everything
not coming from the ABI may be modified in a way that breaks existing code
any day, though.

On 08/04/2013 07:24 PM, Sam Lantinga wrote:

Done!

On Sun, Aug 4, 2013 at 5:23 AM, Alexey Petruchik <i100500 at gmail.com <mailto:i100500 at gmail.com>> wrote:

Although SDL_android.h is not intended to be included by client code
sometimes it needed. For example you need JNIEnv pointer to make JNI
calls to modified SDLActivity.java (video playback, facebook
integration, in-apps). It seems a bit weird to write:

extern "C" {
#include "SDL_android.h"
}

in my AndroidJNI.cpp file.

Regards, Alexey.

______________________________**_________________
SDL mailing list
SDL at lists.libsdl.org <mailto:SDL at lists.libsdl.org>
http://lists.libsdl.org/**listinfo.cgi/sdl-libsdl.org<http://lists.libsdl.org/listinfo.cgi/sdl-libsdl.org>

_____________**
SDL mailing list
SDL at lists.libsdl.org
http://lists.libsdl.org/**listinfo.cgi/sdl-libsdl.orghttp://lists.libsdl.org/listinfo.cgi/sdl-libsdl.org

_____________**
SDL mailing list
SDL at lists.libsdl.org
http://lists.libsdl.org/**listinfo.cgi/sdl-libsdl.orghttp://lists.libsdl.org/listinfo.cgi/sdl-libsdl.org