I believe you confused "added NULL checks" with "removed abort()s"

On Mon, Nov 12, 2012 at 2:45 PM, Enlightenment SVN <
[email protected]> wrote:

> Log:
> more null checks in evas to avoid segs.
>
>
>
> Author:       raster
> Date:         2012-11-12 06:45:07 -0800 (Mon, 12 Nov 2012)
> New Revision: 79167
> Trac:         http://trac.enlightenment.org/e/changeset/79167
>
> Modified:
>   trunk/efl/src/lib/evas/canvas/evas_clip.c
> trunk/efl/src/modules/evas/engines/gl_x11/evas_engine.c
>
> Modified: trunk/efl/src/lib/evas/canvas/evas_clip.c
> ===================================================================
> --- trunk/efl/src/lib/evas/canvas/evas_clip.c   2012-11-12 14:44:30 UTC
> (rev 79166)
> +++ trunk/efl/src/lib/evas/canvas/evas_clip.c   2012-11-12 14:45:07 UTC
> (rev 79167)
> @@ -219,26 +219,22 @@
>     if (clip->delete_me)
>       {
>          CRIT("Setting deleted object %p as clip obj %p", eo_clip, eo_obj);
> -        abort();
>          return;
>       }
>     if (obj->delete_me)
>       {
>          CRIT("Setting object %p as clip to deleted obj %p", eo_clip,
> eo_obj);
> -        abort();
>          return;
>       }
>     if (!obj->layer)
>       {
>          CRIT("No evas surface associated with object (%p)", eo_obj);
> -        abort();
>          return;
>       }
>     if ((obj->layer && clip->layer) &&
>         (obj->layer->evas != clip->layer->evas))
>       {
>          CRIT("Setting object %p from Evas (%p) to another Evas (%p)",
> obj, obj->layer->evas, clip->layer->evas);
> -        abort();
>          return;
>       }
>
>
> Modified: trunk/efl/src/modules/evas/engines/gl_x11/evas_engine.c
> ===================================================================
> --- trunk/efl/src/modules/evas/engines/gl_x11/evas_engine.c     2012-11-12
> 14:44:30 UTC (rev 79166)
> +++ trunk/efl/src/modules/evas/engines/gl_x11/evas_engine.c     2012-11-12
> 14:45:07 UTC (rev 79167)
> @@ -1939,7 +1939,7 @@
>                          ERR("Try glXCreatePixmap on GLX with no support");
>                        if (n->glx_pixmap)
>                          {
> -//                           printf("%p: new native texture for %x | %4i
> x %4i @ %2i = %p\n",
> +//                          printf("%p: new native texture for %x | %4i x
> %4i @ %2i = %p\n",
>  //                                  n, pm, w, h, depth, n->glx_pixmap);
>                             if (!target)
>                               {
>
>
>
> ------------------------------------------------------------------------------
> Everyone hates slow websites. So do we.
> Make your web apps faster with AppDynamics
> Download AppDynamics Lite for free today:
> http://p.sf.net/sfu/appdyn_d2d_nov
> _______________________________________________
> enlightenment-svn mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/enlightenment-svn
>
------------------------------------------------------------------------------
Everyone hates slow websites. So do we.
Make your web apps faster with AppDynamics
Download AppDynamics Lite for free today:
http://p.sf.net/sfu/appdyn_d2d_nov
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to