Closed
Bug 628754
Opened 15 years ago
Closed 15 years ago
Remove unused createUTF8StringEnumerator and its associated enumerator from calUtils
Categories
(Calendar :: Build Config, defect)
Calendar
Build Config
Tracking
(Not tracked)
RESOLVED
FIXED
1.0b3
People
(Reporter: sgautherie, Assigned: standard8)
References
(Blocks 1 open bug, )
Details
Attachments
(1 file)
|
2.41 KB,
patch
|
Fallen
:
review+
|
Details | Diff | Splinter Review |
{
/calendar/base/src/calUtils.h
* line 50 -- #include "nsVoidArray.h"
}
|
Assignee | |
Comment 1•15 years ago
|
||
Altering subject to be more meaningful, but it'll have the same effect. Patch up in a moment.
Summary: Remove nsVoidArray from Calendar → Remove unused createUTF8StringEnumerator and its associated enumerator from calUtils
|
|
Assignee | |
Comment 2•15 years ago
|
||
According to mxr this is completely unused. I haven't test compiled it as I don't have a Lightning build handy, however it should work ;-)
|
||
Comment 3•15 years ago
|
||
Comment on attachment 512145 [details] [diff] [review]
The fix
r=philipp, seems we have switched to a js implementation so removing this is fine :) Also, a=philipp for comm-1.9.2, go ahead and push there too.
Attachment #512145 -
Flags: review?(philipp) → review+
|
|
Assignee | |
Comment 4•15 years ago
|
||
Checked in:
http://hg.mozilla.org/comm-central/rev/54fc62084b8a
http://hg.mozilla.org/releases/comm-1.9.2/rev/5a29790c8023
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → 1.0b3
You need to log in
before you can comment on or make changes to this bug.
Description
•