android_native_app_glue implementation

85 views
Skip to first unread message

simons....@gmail.com

unread,
Jan 26, 2011, 2:03:16 PM1/26/11
to andro...@googlegroups.com
android_native_app_glue.h say that when APP_CMD_TERM_WINDOW is recieved android_app->window still contains the  the existing window.
yet in android_native_app_glue.c void android_app_pre_exec_cmd(struct android_app* android_app, int8_t cmd) function the window is set to NULL.
is this an implementation bug?

TIA

David Turner

unread,
Jan 26, 2011, 6:06:07 PM1/26/11
to andro...@googlegroups.com, Dianne Hackborn
It looks like an implementation bug. android_app_post_exec_cmd sets the field to NULL as you would expect, so I suspect that android_app_pre_exec_cmd should not. Adding Dianne on cc so she can answer this.

Can you file a bug at b.android.com for this. You can also trivially fix your own copy of the source file in the meantime, sorry for that.
 
TIA

--
You received this message because you are subscribed to the Google Groups "android-ndk" group.
To post to this group, send email to andro...@googlegroups.com.
To unsubscribe from this group, send email to android-ndk...@googlegroups.com.
For more options, visit this group at http://groups.google.com/group/android-ndk?hl=en.

Dianne Hackborn

unread,
Jan 26, 2011, 8:08:58 PM1/26/11
to andro...@googlegroups.com
Sorry yes android_app_pre_exec should not clear it.
--
Dianne Hackborn
Android framework engineer
hac...@android.com

Note: please don't send private questions to me, as I don't have time to provide private support, and so won't reply to such e-mails.  All such questions should be posted on public forums, where I and others can see and answer them.

simons....@gmail.com

unread,
Jan 27, 2011, 8:55:45 AM1/27/11
to andro...@googlegroups.com
bug 14352 filled
Reply all
Reply to author
Forward
0 new messages