Has this been reverted already? (The unrelated changes) Or should I do it?
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