On 31/08/16 00:35, Carsten Haitzler (The Rasterman) wrote: > On Tue, 30 Aug 2016 13:37:34 +0100 Tom Hacohen <t...@osg.samsung.com> said: > >> Has this been reverted already? (The unrelated changes) >> >> Or should I do it? > > well the changes there don't hurt anything if you look at them. they would > avoid crashes if null params were passed in so i see removing them as worse > than doing nothing. i have no idea how they got there. i hadn't touched any gl > engine stuff in the few days i was working on this mime stuff.
Ah great, so should be kept. Was just making sure we're not missing on this. -- Tom. > >> On 27/08/16 01:48, Carsten Haitzler (The Rasterman) wrote: >>> On Fri, 26 Aug 2016 14:15:46 +0000 Mike Blumenkrantz >>> <michael.blumenkra...@gmail.com> said: >>> >>>> I just noticed this while doing some bisecting...was it intentional that >>>> you included undocumented evas engine changes in your efreet commit? >>> >>> wtf? i... what? i haven't even opened evas_gl_cocoa_main.m in like... >>> months and months - i think i edited then once before 1.18 release to fix a >>> bug... wtf? how? i didn't touch these! i haven't been looking at anything >>> in evas engines for a while... certainly not int he last week or 2. >>> >>> WTH? >>> >>>> On Tue, Jul 5, 2016 at 3:29 AM Carsten Haitzler <ras...@rasterman.com> >>>> wrote: >>>> >>>>> raster pushed a commit to branch master. >>>>> >>>>> >>>>> http://git.enlightenment.org/core/efl.git/commit/?id=2ca5075193e27dbc30f26e478019719a5360cc84 >>>>> >>>>> commit 2ca5075193e27dbc30f26e478019719a5360cc84 >>>>> Author: Carsten Haitzler (Rasterman) <ras...@rasterman.com> >>>>> Date: Tue Jul 5 15:56:43 2016 +0900 >>>>> >>>>> efreet - fix mime file path messing with short paths like / >>>>> >>>>> this fixes T4015 >>>>> >>>>> @fix >>>>> --- >>>>> src/lib/efreet/efreet_mime.c | 7 +++++-- >>>>> src/modules/evas/engines/gl_cocoa/evas_gl_cocoa_main.m | 1 + >>>>> src/modules/evas/engines/gl_x11/evas_x_main.c | 2 ++ >>>>> src/modules/evas/engines/wayland_egl/evas_wl_main.c | 1 + >>>>> 4 files changed, 9 insertions(+), 2 deletions(-) >>>>> >>>>> diff --git a/src/lib/efreet/efreet_mime.c b/src/lib/efreet/efreet_mime.c >>>>> index adfa790..87ca8ef 100644 >>>>> --- a/src/lib/efreet/efreet_mime.c >>>>> +++ b/src/lib/efreet/efreet_mime.c >>>>> @@ -711,10 +711,13 @@ efreet_mime_special_check(const char *file) >>>>> parent[PATH_MAX - 1] = '\0'; >>>>> >>>>> /* Kill any trailing slash */ >>>>> - parent[--path_len] = '\0'; >>>>> + if (parent[path_len - 1] == '/') >>>>> + parent[--path_len] = '\0'; >>>>> >>>>> /* Truncate to last slash */ >>>>> - while (parent[--path_len] != '/') parent[path_len] = '\0'; >>>>> + while ((path_len > 0) && >>>>> + (parent[--path_len] != '/')) >>>>> + parent[path_len] = '\0'; >>>>> >>>>> #ifdef _WIN32 >>>>> if (!stat(file, &s2)) >>>>> diff --git a/src/modules/evas/engines/gl_cocoa/evas_gl_cocoa_main.m >>>>> b/src/modules/evas/engines/gl_cocoa/evas_gl_cocoa_main.m >>>>> index 15dc218..7df919f 100644 >>>>> --- a/src/modules/evas/engines/gl_cocoa/evas_gl_cocoa_main.m >>>>> +++ b/src/modules/evas/engines/gl_cocoa/evas_gl_cocoa_main.m >>>>> @@ -120,6 +120,7 @@ eng_window_free(Evas_GL_Cocoa_Window *gw) >>>>> void >>>>> eng_window_use(Evas_GL_Cocoa_Window *gw) >>>>> { >>>>> + if ((gw) && (!gw->gl_context)) return; >>>>> if (_evas_gl_cocoa_window != gw) >>>>> { >>>>> [[(NSOpenGLView*)gw->view openGLContext] makeCurrentContext]; >>>>> diff --git a/src/modules/evas/engines/gl_x11/evas_x_main.c >>>>> b/src/modules/evas/engines/gl_x11/evas_x_main.c >>>>> index 59361e1..f5e2edf 100644 >>>>> --- a/src/modules/evas/engines/gl_x11/evas_x_main.c >>>>> +++ b/src/modules/evas/engines/gl_x11/evas_x_main.c >>>>> @@ -676,6 +676,8 @@ eng_window_use(Outbuf *gw) >>>>> xwin = _tls_outbuf_get(); >>>>> >>>>> glsym_evas_gl_preload_render_lock(eng_window_make_current, gw); >>>>> + if ((gw) && (!gw->gl_context)) return; >>>>> + >>>>> #ifdef GL_GLES >>>>> if (xwin) >>>>> { >>>>> diff --git a/src/modules/evas/engines/wayland_egl/evas_wl_main.c >>>>> b/src/modules/evas/engines/wayland_egl/evas_wl_main.c >>>>> index 489a475..14e13ff 100644 >>>>> --- a/src/modules/evas/engines/wayland_egl/evas_wl_main.c >>>>> +++ b/src/modules/evas/engines/wayland_egl/evas_wl_main.c >>>>> @@ -233,6 +233,7 @@ eng_window_use(Outbuf *gw) >>>>> Eina_Bool force = EINA_FALSE; >>>>> >>>>> glsym_evas_gl_preload_render_lock(eng_preload_make_current, gw); >>>>> + if ((gw) && (!gw->gl_context)) return; >>>>> >>>>> if (_evas_gl_wl_window) >>>>> { >>>>> >>>>> -- >>>>> >>>>> >>>>> >>>> ------------------------------------------------------------------------------ >>>> _______________________________________________ >>>> enlightenment-devel mailing list >>>> enlightenment-devel@lists.sourceforge.net >>>> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel >>>> >>> >>> >> >> >> ------------------------------------------------------------------------------ >> _______________________________________________ >> enlightenment-devel mailing list >> enlightenment-devel@lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel >> > > ------------------------------------------------------------------------------ _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel