Re: PSA: mozilla::Result is about to understand move semantics

2019-08-13 Thread Kris Maglione
Just for some additional extra context... The C++ mozilla::Result implementation is pretty heavily optimized for common use cases to try to pack its representation into a single machine word when possible. For instance, if you return a `Result`, you're really just returning an `nsresult`. If

Re: PSA: mozilla::Result is about to understand move semantics

2019-08-13 Thread Alexis Beingessner
Just chiming in here with some brief extra context on the performance of Result (which I really need to do a writeup about so that I can just link it): TL;DR performance of Result is usually fine, but can randomly be a huge problem. However there's also improvements for this on the (distant)

Re: PSA: mozilla::Result is about to understand move semantics

2019-08-13 Thread Kris Maglione
On Mon, Aug 12, 2019 at 10:14:19PM -0700, Bryce Seager van Dyk wrote: But either way, that's going to result in a copy when the Result is constructed (unless the compiler is really clever). Is it the data being moved into the Result which is incurring the copy here, or the actual Result

Re: Documentation for IPDL needs some edit

2019-08-13 Thread saschanaz7
Okay, thanks for the confirmation! Will do some needed edits. ___ dev-platform mailing list dev-platform@lists.mozilla.org https://lists.mozilla.org/listinfo/dev-platform

Intent to unship: 's mode attribute

2019-08-13 Thread Frédéric Wang
Hi, In bug 1573438, I intend to remove support for the mode attribute on the element. This attribute is from MathML 1 and has been deprecated from MathML 2 (released 16 years ago) in favor of the display attribute [1] and we have been sending deprecation warnings since Mozilla 20 (released 6

Re: Removing --enable-shared-js [Was: Rust and --enable-shared-js]

2019-08-13 Thread Lars Hansen
Cranelift should be genuinely optional until further notice; to my knownledge, no near-term product work in Firefox or SpiderMonkey depends on Cranelift. Cranelift is present in Nightly but (so far as I can tell) not in Release. It can be disabled in the JS shell by configuring with