On 02/08/15 05:57 AM, Kyle Huey wrote: > On Sun, Aug 2, 2015 at 2:56 AM, Hubert Figuière <h...@mozilla.com> wrote: >> On 02/08/15 04:55 AM, smaug wrote: >>> A new type of error 'auto' seems to cause, now seen on m-i, is >>> auto foo = new SomeRefCountedFoo(); >>> >>> That hides that foo is a raw pointer but we should be using nsRefPtr. >>> >>> So please, consider again when about to use auto. It usually doesn't >>> make the code easier to read, >>> and it occasionally just leads to errors. In this case it clearly made >>> the code harder to read so that >>> whoever reviewed that patch didn't catch the issue. >> >> Shouldn't we, instead, ensure that SomeRefCountedFoo() returns a nsRefPtr? > > How do you do that with a constructor?
Uh oh, my bad. As suggested by Xidorn, having a construction method (class static) and making the constructor protected, is the right way to do it. Hub _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform
