Closed Bug 1588260 Opened 9 months ago Closed 8 months ago

Stop using [array] in nsICollation

Categories

(Core :: Internationalization, task, P3)

task

Tracking

()

RESOLVED FIXED
mozilla72
Tracking Status
firefox72 --- fixed

People

(Reporter: bzbarsky, Assigned: bzbarsky)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

I considered using Span here, but nsTArray actually works fine for the one consumer of these APIs in Gecko...

Priority: -- → P3

Magnus, any objections to me landing this on m-c? Are the Thunderbird bits that are affected by this ready to go?

Flags: needinfo?(mkmelin+mozilla)

I'll have to defer to Ben. Nice to see it happening!

Flags: needinfo?(mkmelin+mozilla) → needinfo?(benc)

Thanks, Boris! I got a little sidetracked over the last couple of weeks, but I'll get the Thunderbird side ready to land this week.

Flags: needinfo?(benc)
Blocks: 1593582

Sorry for the delay, Boris, I think we're good to go.

Pushed by bzbarsky@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/dfa02c90256c
part 1.  Make some XSLT comparator code a bit more type-safe.  r=peterv.
https://hg.mozilla.org/integration/autoland/rev/d9acb8da0768
part 2.  Switch nsICollation away from using xpidl [array].  r=peterv,emk
Status: NEW → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
You need to log in before you can comment on or make changes to this bug.