Closed Bug 1706894 Opened 4 years ago Closed 4 years ago

Avoid unnecessary copies in `mozInlineSpellWordUtil`

Categories

(Core :: Spelling checker, enhancement)

Product:
Core
Component:
Spelling checker
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
firefox90 --- fixed

People

(Reporter: mbrodesser, Assigned: mbrodesser)

References

Details

Attachments

(3 files)

Bug 1706894: part 1) Use `std::move` in `DOMTextMapping`'s constructor. r=smaug
48 bytes, text/x-phabricator-request
Details | Review
Bug 1706894: part 2) In `NodeOffsetRange` use `std::move` and return `const` references instead of copies. r=smaug
48 bytes, text/x-phabricator-request
Details | Review
Bug 1706894: part 3) Change `MapDOMPositionToSoftTextOffset`'s argument to `const` reference. r=smaug
48 bytes, text/x-phabricator-request
Details | Review
No description provided.
Pushed by mbrodesser@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/9f1a803302bf part 1) Use `std::move` in `DOMTextMapping`'s constructor. r=smaug https://hg.mozilla.org/integration/autoland/rev/680b3d4a404a part 2) In `NodeOffsetRange` use `std::move` and return `const` references instead of copies. r=smaug https://hg.mozilla.org/integration/autoland/rev/e24868ec9d87 part 3) Change `MapDOMPositionToSoftTextOffset`'s argument to `const` reference. r=smaug
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
status-firefox90: --- → fixed
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: