We pass T* as an argument to a function too often for this to e practical. The best solution is probably to remove the conversion from RefPtr<T>&& to T* which is I believe what froydnj is planning to do.
This requires ref qualifiers for methods, which isn't supported in MSVC 2013, but is supported in 2015. See https://bugzilla.mozilla.org/show_bug.cgi?id=1280295 On Thu, Jun 16, 2016 at 12:27 PM, Eric Rescorla <e...@rtfm.com> wrote: > Is it worth reconsidering removing implicit conversion from RefPtr<T> to > T*? > > -Ekr > > > On Thu, Jun 16, 2016 at 9:50 AM, Boris Zbarsky <bzbar...@mit.edu> wrote: > > > On 6/16/16 3:15 AM, jww...@mozilla.com wrote: > > > >> I think that is the legacy when we don't have move semantics. Returning > >> RefPtr<t> won't incur ref-counting overhead and is more expressive and > >> functional. > >> > > > > Except for the footgun described in < > > https://bugzilla.mozilla.org/show_bug.cgi?id=1280296#c0>, yes? > > > > -Boris > > > > > > _______________________________________________ > > dev-platform mailing list > > dev-platform@lists.mozilla.org > > https://lists.mozilla.org/listinfo/dev-platform > > > _______________________________________________ > dev-platform mailing list > dev-platform@lists.mozilla.org > https://lists.mozilla.org/listinfo/dev-platform > _______________________________________________ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform
