On Tue, 14 Feb 2017 07:10:44 -0500 Christopher Michael <[email protected]>
said:

> Apologize for the incorrect fix there Jean-Philippe, hope I did not 
> break things too badly for you :)

that's ok. the code was misleading. it indented to do X but was structured to
do Y... it's a fair enough confusion and one of these has to win out as the
correct path. :)

> Cheers,
> dh
> 
> On 02/13/2017 09:13 PM, Jean-Philippe ANDRÉ wrote:
> > jpeg pushed a commit to branch master.
> >
> > http://git.enlightenment.org/core/efl.git/commit/?id=45d846814f813b8575276edf0ee9586d7cd25ecc
> >
> > commit 45d846814f813b8575276edf0ee9586d7cd25ecc
> > Author: Jean-Philippe Andre <[email protected]>
> > Date:   Tue Feb 14 10:54:32 2017 +0900
> >
> >     evas gl: Fix memory tracking
> >
> >     See b16c961680579ad13 which by "fixing" indentation changed the
> >     logic of the code. Yes the indentation was misleading (because of
> >     the #if / #else) but the logic was correct. Anyway this is was not
> >     a harmful issue as those memory trackers aren't really used.
> > ---
> >  src/modules/evas/engines/gl_common/evas_gl_core.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/src/modules/evas/engines/gl_common/evas_gl_core.c
> > b/src/modules/evas/engines/gl_common/evas_gl_core.c index 9eb4c0d..b50adfc
> > 100644
> > --- a/src/modules/evas/engines/gl_common/evas_gl_core.c
> > +++ b/src/modules/evas/engines/gl_common/evas_gl_core.c
> > @@ -1344,8 +1344,8 @@ _surface_buffers_allocate(void *eng_data EINA_UNUSED,
> > EVGL_Surface *sfc, int w, _renderbuffer_allocate(sfc->depth_stencil_buf,
> >                                      sfc->depth_stencil_fmt, w, h,
> >                                      sfc->msaa_samples);
> > -             sfc->buffer_mem[3] = w * h * 4;
> >            }
> > +        sfc->buffer_mem[3] = w * h * 4;
> >       }
> >     else
> >       {
> >
> 
> 
> 
> ------------------------------------------------------------------------------
> Check out the vibrant tech community on one of the world's most
> engaging tech sites, SlashDot.org! http://sdm.link/slashdot
> _______________________________________________
> enlightenment-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/enlightenment-devel


-- 
------------- Codito, ergo sum - "I code, therefore I am" --------------
The Rasterman (Carsten Haitzler)    [email protected]


------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to