Closed
Bug 1466297
Opened 7 years ago
Closed 7 years ago
Port bug 1465585 - mailnews/base/search/src/nsMsgFilterList.cpp:499:37: error: 'Move' is not a member of 'mozilla'
Categories
(MailNews Core :: General, defect)
MailNews Core
General
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 62.0
People
(Reporter: jorgk-bmo, Assigned: jorgk-bmo)
Details
Attachments
(1 file)
|
1.06 KB,
patch
|
aceman
:
review+
|
Details | Diff | Splinter Review |
We use mozille::Move() once in all of M-C and that code is:
nsresult nsMsgFilterList::LoadTextFilters(already_AddRefed<nsIInputStream> aStream)
{
nsresult err = NS_OK;
uint64_t bytesAvailable;
nsCOMPtr<nsIInputStream> bufStream;
nsCOMPtr<nsIInputStream> stream = mozilla::Move(aStream);
err = NS_NewBufferedInputStream(getter_AddRefs(bufStream), stream.forget(), FILE_IO_BUFFER_SIZE);
NS_ENSURE_SUCCESS(err, err);
| Assignee | ||
Comment 1•7 years ago
|
||
Found it:
b54db6622358 Emilio Cobos Álvarez — Bug 1465585: Switch from mozilla::Move to std::move. r=froydnj
Summary: Port bug xxx? - mailnews/base/search/src/nsMsgFilterList.cpp:499:37: error: 'Move' is not a member of 'mozilla' → Port bug 1465585 - mailnews/base/search/src/nsMsgFilterList.cpp:499:37: error: 'Move' is not a member of 'mozilla'
| Assignee | ||
Comment 2•7 years ago
|
||
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/16e80af1dc4a
Port bug 1465585: Switch from mozilla::Move to std::move. rs=bustage-fix
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
| Assignee | ||
Updated•7 years ago
|
Target Milestone: --- → Thunderbird 62.0
Comment on attachment 8982765 [details] [diff] [review]
1465585-move.patch
Review of attachment 8982765 [details] [diff] [review]:
-----------------------------------------------------------------
Thanks.
Attachment #8982765 -
Flags: review?(acelists) → review+
You need to log in
before you can comment on or make changes to this bug.
Description
•