Closed Bug 1288077 Opened 3 years ago Closed 3 years ago

Minor cleanups to nsTArray.h and callers

Categories

(Core :: XPCOM, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla50
Tracking Status
firefox50 --- fixed

People

(Reporter: tzimmermann, Assigned: tzimmermann)

Details

Attachments

(4 files)

No description provided.
Review commit: https://reviewboard.mozilla.org/r/65566/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/65566/
Attachment #8772850 - Flags: review?(nfroyd)
Attachment #8772851 - Flags: review?(nfroyd)
Attachment #8772852 - Flags: review?(nfroyd)
Attachment #8772853 - Flags: review?(nfroyd)
The two implemenations of |nsTArrayFallibleAllocator| in 'nsTArray' are
equivalent. This patch merges both.

Review commit: https://reviewboard.mozilla.org/r/65572/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/65572/
Attachment #8772850 - Flags: review?(nfroyd) → review+
Comment on attachment 8772851 [details]
Bug 1288077: Forward declare arrays by including 'nsTArrayForwardDeclare.h',

https://reviewboard.mozilla.org/r/65568/#review62532
Attachment #8772851 - Flags: review?(nfroyd) → review+
Comment on attachment 8772852 [details]
Bug 1288077: Cleanup forward declarations of array types,

https://reviewboard.mozilla.org/r/65570/#review62534
Attachment #8772852 - Flags: review?(nfroyd) → review+
Comment on attachment 8772853 [details]
Bug 1288077: Merge implementations of |nsTArrayFallibleAllocator|,

https://reviewboard.mozilla.org/r/65572/#review62536
Attachment #8772853 - Flags: review?(nfroyd) → review+
Pushed by tdz@users.sourceforge.net:
https://hg.mozilla.org/integration/mozilla-inbound/rev/3158fb3b7e23
Whitespace fixes, r=froydnj
https://hg.mozilla.org/integration/mozilla-inbound/rev/73c97e5ff2f8
Forward declare arrays by including 'nsTArrayForwardDeclare.h', r=froydnj
https://hg.mozilla.org/integration/mozilla-inbound/rev/156883ff3dda
Cleanup forward declarations of array types, r=froydnj
https://hg.mozilla.org/integration/mozilla-inbound/rev/234f9780a4cd
Merge implementations of |nsTArrayFallibleAllocator|, r=froydnj
You need to log in before you can comment on or make changes to this bug.