I agree on you.
Current vector is not fully stable.
And I don't want crashes in vector usage.

Subhransu knows the problem and keeping look the issue.
Once it's fixed, he will remove it.

------------------------------------
-Regards, Hermet-

-----Original Message-----
From: "Tom Hacohen"<t...@osg.samsung.com> 
To: "Enlightenment developer list"<enlightenment-devel@lists.sourceforge.net>; 
Cc: 
Sent: 2015-05-08 (금) 20:44:30
Subject: Re: [E-devel] [EGIT] [core/efl] master 02/02: evas/canvas: Null check 
for base class data pointer.
 
This is pretty much not the way to go. We don't want to hide issues, we 
want to fix them. You just fixed a very small symptom. I don't see a 
ticket on phab, I don't see a report, I don't see anything. This is bad.

--
Tom.

On 08/05/15 12:40, Subhransu Mohanty wrote:
> hermet pushed a commit to branch master.
>
> http://git.enlightenment.org/core/efl.git/commit/?id=8f55ac7550360e6afcdc0567685ceb25b7ecec63
>
> commit 8f55ac7550360e6afcdc0567685ceb25b7ecec63
> Author: Subhransu Mohanty <sub.moha...@samsung.com>
> Date:   Fri May 8 20:32:40 2015 +0900
>
>      evas/canvas: Null check for base class data pointer.
> ---
>   src/lib/evas/canvas/evas_object_vg.c  4 ++++
>   1 file changed, 4 insertions(+)
>
> diff --git a/src/lib/evas/canvas/evas_object_vg.c 
> b/src/lib/evas/canvas/evas_object_vg.c
> index 57d4090..cca772a 100644
> --- a/src/lib/evas/canvas/evas_object_vg.c
> +++ b/src/lib/evas/canvas/evas_object_vg.c
> @@ -226,6 +226,10 @@ evas_object_vg_render_pre(Evas_Object *eo_obj,
>      // FIXME: for now the walking Evas_VG_Node tree doesn't trigger any 
> damage
>      // So just forcing it here if necessary
>      rnd = eo_data_scope_get(vd->root, EFL_VG_BASE_CLASS);
> +
> +   //FIXME find the reason for NULL Base Class in some case?
> +   if (!rnd) return;
> +
>      if (rnd->changed)
>        {
>           rnd->changed = EINA_FALSE;
>


------------------------------------------------------------------------------
One dashboard for servers and applications across Physical-Virtual-Cloud 
Widest out-of-the-box monitoring support with 50+ applications
Performance metrics, stats and reports that give you Actionable Insights
Deep dive visibility with transaction tracing using APM Insight.
http://ad.doubleclick.net/ddm/clk/290420510;117567292;y
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
------------------------------------------------------------------------------
One dashboard for servers and applications across Physical-Virtual-Cloud 
Widest out-of-the-box monitoring support with 50+ applications
Performance metrics, stats and reports that give you Actionable Insights
Deep dive visibility with transaction tracing using APM Insight.
http://ad.doubleclick.net/ddm/clk/290420510;117567292;y
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to