[02:04:10] --- DominusExult is now known as Dominus
[08:52:12] <Dominus> wjp, if I exchange the order of SDL_DestroyRenderer and SDL_DestroyTexture at https://github.com/litchie/exult-ios/blob/master/imagewin/imagewin.cc#L666 all is fine
[08:54:22] <Dominus> safe to do that?
[08:57:53] <Dominus> quick glance shows that some other projects do it in this order
[09:08:09] <wjp> I have no idea... the online docs on these functions are rather sparse
[09:08:19] <wjp> but it's plausible
[09:08:30] <wjp> oh wait, DestroyRenderer does have a relevant comment
[09:08:42] <wjp> "Use this function to destroy the rendering context for a window and free associated textures."
[09:08:59] <wjp> so swapping the order should be good
[09:09:04] <wjp> nice catch
[09:09:38] <wjp> (note that there's another spot with the same two calls in the same order)
[09:09:48] <Dominus> things that drive you mad and make you insane things with imagewin.cc until you find the right spot :)
[09:09:57] <Dominus> yes, I saw that
[09:10:14] <Dominus> So, it makes sense to swap this for main Exult as well
[09:10:42] <Dominus> I'm still puzzled why it doesn't trigger on desktop machines...
[09:12:37] <Dominus> something I'm going to try next on my mac with a if screen_texture != NULL -> stdout something after DestroyRenderer
[09:12:57] <Dominus> Thanks for looking with me! :)
[09:36:41] <Dominus> Imagine the debugger as a person who screams at you WHAT ARE YOU DOING IN THOSE OTHER FUNCTIONS? THE PROBLEM IS RIGHT HERE WHERE I POINT YOU AT!!!
[10:40:49] <Dominus> hmm, odd the value of screen_texture is not changing after DestroyRenderer but oh well, somewhere something is...
[12:04:50] <wjp> there's no reason it would change
[14:34:28] --- ttarrant_ is now known as ttarrant
[19:42:32] <Dominus> Wheee! My very first patch accepted into SDL2! (With some modifications, so not completely clean, but still ;))
[19:55:36] <Marzo> Dominus: congrats
[19:55:48] <Marzo> Was it for that texture issue?
[19:56:10] <Marzo> Or something else?
[19:58:59] <Dominus> Marzo: something else.
[19:59:35] <Dominus> https://hg.libsdl.org/SDL/rev/98ea6e4c0d68
[19:59:48] <Dominus> Trivial but annoying
[20:01:39] <Dominus> The texture issue I will ask in the forum why it asserts on ios but not on Desktop