On Thu, Jun 16, 2016 at 2:15 PM, Michael Layzell <[email protected]> wrote:
> We pass T* as an argument to a function too often for this to e practical. > Can you explain why? Is your point here to avoid people having to type .get() or the mass conversion from the current code? The former seems like it's a feature rather than a bug... -Ekr 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 <[email protected]> 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 <[email protected]> wrote: >> >> > On 6/16/16 3:15 AM, [email protected] 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 >> > [email protected] >> > https://lists.mozilla.org/listinfo/dev-platform >> > >> _______________________________________________ >> dev-platform mailing list >> [email protected] >> https://lists.mozilla.org/listinfo/dev-platform >> > > _______________________________________________ dev-platform mailing list [email protected] https://lists.mozilla.org/listinfo/dev-platform
