Closed Bug 1435924 Opened 2 years ago Closed 2 years ago
nsSubstring.h just includes nsAString.h, we can remove it and use nsAString.h where appropriate. :m_kato tried this 6 years ago in bug 737400 and it was rejected due to nsSubstring being "the preferred name for the base string class." I think the ship has sailed on that, we only us nsAC?String now for our interfaces and nothing is actually including 'nsSubstring.h' outside of xpcom/string.
Attachment #8948594 - Flags: review?(dbaron) → review+
Attachment #8948595 - Flags: review?(dbaron) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/e945fa4f3c1f Part 1: Switch from nsSubstring.h to nsAString.h. r=dbaron https://hg.mozilla.org/integration/mozilla-inbound/rev/352ef85382ea Part 2: Remove nsSubstring.h. r=dbaron
You need to log in before you can comment on or make changes to this bug.