On Wed, 11 Dec 2013 13:55:52 -0200 Gustavo Sverzut Barbieri
<barbi...@gmail.com> said:

> On Wed, Dec 11, 2013 at 8:43 AM, Carsten Haitzler <ras...@rasterman.com>
> wrote:
> > raster pushed a commit to branch master.
> >
> > http://git.enlightenment.org/core/efl.git/commit/?id=f779c49c89875115a0b715a05c524bbb3ed6556c
> >
> > commit f779c49c89875115a0b715a05c524bbb3ed6556c
> > Author: Carsten Haitzler (Rasterman) <ras...@rasterman.com>
> > Date:   Wed Dec 11 18:20:13 2013 +0900
> >
> >     edje - edje_cc - try and make coverty ignore the leak (invalid)
> >
> >     hope that this does not reappear in future if changes happen in src.
> >     CID 1132627
> > ---
> >  src/bin/edje/edje_cc_out.c | 7 +++++--
> >  1 file changed, 5 insertions(+), 2 deletions(-)
> >
> > diff --git a/src/bin/edje/edje_cc_out.c b/src/bin/edje/edje_cc_out.c
> > index f5a5044..632a8bb 100644
> > --- a/src/bin/edje/edje_cc_out.c
> > +++ b/src/bin/edje/edje_cc_out.c
> > @@ -945,8 +945,11 @@ data_write_images(Eet_File *ef, int *image_num)
> >                           data_image_preload_done(iw, evas, im, NULL);
> >                      }
> >                    else
> > -                    error_and_abort_image_load_error
> > -                    (ef, img->entry, load_err);
> > +                    {
> > +                       error_and_abort_image_load_error
> > +                         (ef, img->entry, load_err);
> > +                       exit(1); // ensure static analysis tools know we
> > exit
> > +                    }
> 
> You could mark the function with the __attribute__(noreturn)

i know.. but then does coverty know that. i'm going through coverty scan stuff.
i did it in a way i am more certain it will figure out. i can't ASK it to
re-scan. it scans once per day. so i have to take a stab and see. and commits
have to go in first for it to scan them.

-- 
------------- Codito, ergo sum - "I code, therefore I am" --------------
The Rasterman (Carsten Haitzler)    ras...@rasterman.com


------------------------------------------------------------------------------
Rapidly troubleshoot problems before they affect your business. Most IT 
organizations don't have a clear picture of how application performance 
affects their revenue. With AppDynamics, you get 100% visibility into your 
Java,.NET, & PHP application. Start your 15-day FREE TRIAL of AppDynamics Pro!
http://pubads.g.doubleclick.net/gampad/clk?id=84349831&iu=/4140/ostg.clktrk
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to