Closed Bug 737400 Opened 8 years ago Closed 8 years ago

Remove nsSubstring.h

Categories

(Core :: String, defect)

defect
Not set

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: 8 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?
You need to log in before you can comment on or make changes to this bug.