1155457 - nsTString classes don't support move semantics

Status: NEW

(Core :: XPCOM, defect)

Description

[Seth Fowler [:seth] [:s2h]]

We should make nsTString classes support move semantics; they're used all over the place.

Comment 1

[Ben Turner (not reading bugmail, use the needinfo flag!)]

Under the hood they do some very similar stuff by passing refcounted nsStringBuffer around. I guess we could save an AddRef/Release pair in most cases?

Comment 2

[Seth Fowler [:seth] [:s2h]]

(In reply to Ben Turner [:bent] (use the needinfo flag!) from comment #1)
> Under the hood they do some very similar stuff by passing refcounted
> nsStringBuffer around. I guess we could save an AddRef/Release pair in most
> cases?

Yeah, seems like a small but probably still worthwhile win.

There's also a semantic difference between move and copy which usefully clarifies the intent of the code, I think.