Re: [Crystal-cvs] SF.net SVN: crystal:[38283] CS/branches/soc2012/lighter2/apps/tools/ lighter2

On 06.07.2012 18:25, [email protected] wrote:
> -    RadMaterial() {refractiveIndex = 1; isTexImageValid=false; produceCaustic= false;}
> +    RadMaterial() {
> +      refractiveIndex = 1; isTexImageValid=false; produceCaustic= false;
> +      lockMutex = new CS::Threading::Mutex();
> +    }

1. lockMutex leaks.
2. Why allocate it on the heap anyway? “mutable CS::Threading::Mutex
lockMutex;” (no pointer) should work fine.

-f.r.

------------------------------------------------------------------------------
Live Security Virtual Conference
Exclusive live event will cover all the ways today's security and 
threat landscape has changed and how IT managers can respond. Discussions 
will include endpoint security, mobile security and the latest in malware 
threats. http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/

_______________________________________________
Crystal-develop mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/crystal-develop

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.12 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iEYEARECAAYFAk/3qvIACgkQNCTzcE/07ezSMwCgieyUaPUd5+m8Uu36JVXaMJWv
RLcAn1/U5PqnhHGcjwrVjW6O0JMc3AoN
=/L5n
-----END PGP SIGNATURE-----