On Fri, Jan 25, 2013 at 6:42 AM, Enlightenment SVN
<no-re...@enlightenment.org> wrote:
> Log:
> efl: more valgrind related fixes on eina_cow

Ah ! I always want to think that valgrind define read only memory...
Anyway thanks, that explain the issue that raster and Daniel saw I
guess.

> Author:       billiob
> Date:         2013-01-24 13:42:18 -0800 (Thu, 24 Jan 2013)
> New Revision: 83311
> Trac:         http://trac.enlightenment.org/e/changeset/83311
>
> Modified:
>   trunk/efl/src/lib/eina/eina_cow.c
>
> Modified: trunk/efl/src/lib/eina/eina_cow.c
> ===================================================================
> --- trunk/efl/src/lib/eina/eina_cow.c   2013-01-24 21:35:06 UTC (rev 83310)
> +++ trunk/efl/src/lib/eina/eina_cow.c   2013-01-24 21:42:18 UTC (rev 83311)
> @@ -341,12 +341,12 @@
>     VALGRIND_MAKE_MEM_DEFINED(ref, sizeof (ref));
>  #endif
>     ref->refcount--;
> +
> +   if (ref->refcount > 0) return ;
>  #ifndef NVALGRIND
>     VALGRIND_MAKE_MEM_NOACCESS(ref, sizeof (ref));
>  #endif
>
> -   if (ref->refcount > 0) return ;
> -
>  #ifndef NVALGRIND
>     VALGRIND_MAKE_MEM_DEFINED(ref, sizeof (ref));
>  #endif
>
>
> ------------------------------------------------------------------------------
> Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS,
> MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current
> with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft
> MVPs and experts. ON SALE this month only -- learn more at:
> http://p.sf.net/sfu/learnnow-d2d
> _______________________________________________
> enlightenment-svn mailing list
> enlightenment-...@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/enlightenment-svn
>



--
Cedric BAIL

------------------------------------------------------------------------------
Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS,
MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current
with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft
MVPs and experts. ON SALE this month only -- learn more at:
http://p.sf.net/sfu/learnnow-d2d
_______________________________________________
enlightenment-devel mailing list
enlightenment-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel

Reply via email to