1053420 - Improve new nsTArray rvalue reference methods

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[[e-mail]]

The new rvalue reference versions of nsTArray::AppendElement and InsertElementAt want to move their parameter to the Construct function. Since this isn't a template-deduced reference parameter, they should Move rather than Forward.

It looks as if jrmuizel did originally try to template on the parameter to AppendElement but was just trying various combinations until it compiled :-\

Note that template<class Item> elem_type* AppendElement(elem_type&& aItem) matches non-const lvalues which is why you can't blindly Move them. In fact it quite happily takes const lvalues too, making the other overload redundant.

Comment 1

[[e-mail]]

My second try run (the first got hit by our infra issues) for my suggested AppendElement change seems to be reasonably green. Patch is at https://hg.mozilla.org/try/rev/9922524dc8dd

Comment 2

[[e-mail]]

Attachment: Proposed patch

https://tbpl.mozilla.org/?tree=Try&rev=34a6b9a59e6d

Comment 3

[[e-mail]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/2976adaf3278

Comment 4

[[e-mail]]

I botched the bug# on my checkin comment. Oops.

https://hg.mozilla.org/mozilla-central/rev/2976adaf3278