Closed Bug 737400 Opened 9 years ago Closed 9 years ago

Remove nsSubstring.h

Categories

(Core :: XPCOM, defect)

defect
Not set
normal

Tracking

()

RESOLVED INVALID

People

(Reporter: m_kato, Assigned: m_kato)

Details

Attachments

(1 file)

Attached patch fixSplinter Review
nsSubstring.h is for old comptibility.  We should remove this.

Also some headers use nsSubstring_h__ that isn't defined into nsSubstring.h.  If I cannot remove nsSubstring.h for some reasons, I need fix it.
Attachment #607524 - Flags: review?(dbaron)
No it's not -- nsSubstring and  the preferred name for the base string class -- nsAString is a compatibility alias for nsSubstring.
Attachment #607524 - Flags: review?(dbaron) → review-
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → INVALID
(In reply to David Baron [:dbaron] from comment #1)
> No it's not -- nsSubstring and  the preferred name for the base string class
> -- nsAString is a compatibility alias for nsSubstring.

dbaron, nsSubstring_h__ isn't defined into nsSubstring.h.  But some headers use it.  Should I file a new bug?
Component: String → XPCOM
You need to log in before you can comment on or make changes to this bug.