763873 - avoid repeatedly accessing aRanges.Length() within ComputeRanges

Status: RESOLVED FIXED

(Core :: Graphics: Text, defect)

Description

[Jonathan Kew [:jfkthame]]

Attachment: minor optimization

A little micro-optimization.... it's potentially inefficient to constantly read aRanges.Length(); instead, we can keep track of the last range index in a local variable. We also don't need the initial Clear(), as ComputeRanges is always called with a freshly-constructed array.

Comment 1

[Jonathan Kew [:jfkthame]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/064319559cb2

Comment 2

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/064319559cb2