Closed
Bug 737400
Opened 12 years ago
Closed 12 years ago
Remove nsSubstring.h
Categories
(Core :: XPCOM, defect)
Core
XPCOM
Tracking
()
RESOLVED
INVALID
People
(Reporter: m_kato, Assigned: m_kato)
Details
Attachments
(1 file)
6.02 KB,
patch
|
dbaron
:
review-
|
Details | Diff | Splinter 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.
Assignee | ||
Updated•12 years ago
|
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: 12 years ago
Resolution: --- → INVALID
Assignee | ||
Comment 2•12 years ago
|
||
(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?
Updated•3 years ago
|
Component: String → XPCOM
You need to log in
before you can comment on or make changes to this bug.
Description
•