Closed Bug 693949 Opened 9 years ago Closed 9 years ago

Drop nsCStringArray

Categories

(Core :: XPCOM, defect)

Product:
Core
Component:
XPCOM
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla10

People

(Reporter: standard8, Assigned: standard8)

References

Details

Attachments

(1 file)

The fix
13.54 KB, patch
benjamin
: review+
Details | Diff | Splinter Review 
Bug 693946 will remove the last uses of nsCStringArray from the code base, as soon as that lands we can just drop nsCStringArray completely:

http://mxr.mozilla.org/comm-central/search?string=nsCStringArray

I'm just confirming the patch I've done works and will post it in a bit.
Attached patch The fix — — Splinter Review 
This removes the now redundant nsCStringArray. I've taken the opportunity to do some header include cleanup which we'd need to do when we get around to removing nsVoidArray completely.

This passes on try server, and I'll do another check before landing it.
Attachment #568960 - Flags: review?(benjamin)
Comment on attachment 568960 [details] [diff] [review]
The fix

gogogo!
Attachment #568960 - Flags: review?(benjamin) → review+
https://hg.mozilla.org/mozilla-central/rev/fcf252e6ce4a
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla10
You need to log in before you can comment on or make changes to this bug.