Closed
Bug 1417328
Opened 7 years ago
Closed 7 years ago
mailnews should not include nsIPlatformCharset
Categories
(MailNews Core :: Backend, defect)
MailNews Core
Backend
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 59.0
People
(Reporter: m_kato, Assigned: jorgk-bmo)
References
Details
Attachments
(1 file)
|
1.65 KB,
patch
|
Details | Diff | Splinter Review |
Gecko will remove nsIPlatformCharset interface. But mailnews still includes nsIPlatformCharset.h even if it doesn't use this interface. Please remove this include.
https://dxr.mozilla.org/comm-central/search?q=nsIPlatformCharset&redirect=false
mailnews/base/util/nsMsgUtils.cpp
71 #include "nsIPlatformCharset.h"
mailnews/import/src/nsImportService.cpp
7 #include "nsIPlatformCharset.h"
|
Reporter | |
Updated•7 years ago
|
Summary: mailnews should not include ns → mailnews should not include nsIPlatformCharset
|
Assignee | |
Comment 1•7 years ago
|
||
Thanks for the heads-up!
Assignee: nobody → jorgk
Status: NEW → ASSIGNED
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/d08d18c8b791
Remove unnecessary include of soon to be removed nsIPlatformCharset. r=me
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 3•7 years ago
|
||
Need to fix target to TB 59.
Target Milestone: --- → Thunderbird 58.0
|
|
Assignee | |
Updated•7 years ago
|
Target Milestone: Thunderbird 58.0 → Thunderbird 59.0
You need to log in
before you can comment on or make changes to this bug.
Description
•