summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--db/mork/public/mdb.h8
-rw-r--r--db/mork/src/morkRowCellCursor.cpp36
-rw-r--r--db/mork/src/morkRowCellCursor.h8
-rw-r--r--mailnews/import/outlook/src/MapiApi.cpp9
-rw-r--r--mailnews/mime/emitters/nsEmitterUtils.cpp1
-rw-r--r--mailnews/mime/src/mimemsig.cpp1
6 files changed, 3 insertions, 60 deletions
diff --git a/db/mork/public/mdb.h b/db/mork/public/mdb.h
index 9a48434ebb..54d5ac6379 100644
--- a/db/mork/public/mdb.h
+++ b/db/mork/public/mdb.h
@@ -2305,14 +2305,6 @@ public:
NS_IMETHOD GetRow(nsIMdbEnv* ev, nsIMdbRow** acqRow) = 0;
// } ----- end attribute methods -----
- // { ----- begin cell creation methods -----
- NS_IMETHOD MakeCell( // get cell at current pos in the row
- nsIMdbEnv* ev, // context
- mdb_column* outColumn, // column for this particular cell
- mdb_pos* outPos, // position of cell in row sequence
- nsIMdbCell** acqCell) = 0; // the cell at inPos
- // } ----- end cell creation methods -----
-
// { ----- begin cell seeking methods -----
NS_IMETHOD SeekCell( // same as SetRow() followed by MakeCell()
nsIMdbEnv* ev, // context
diff --git a/db/mork/src/morkRowCellCursor.cpp b/db/mork/src/morkRowCellCursor.cpp
index d5d4a0d203..0649b5b551 100644
--- a/db/mork/src/morkRowCellCursor.cpp
+++ b/db/mork/src/morkRowCellCursor.cpp
@@ -183,42 +183,6 @@ morkRowCellCursor::GetRow(nsIMdbEnv* mev, nsIMdbRow** acqRow)
}
// } ----- end attribute methods -----
-// { ----- begin cell creation methods -----
-NS_IMETHODIMP
-morkRowCellCursor::MakeCell( // get cell at current pos in the row
- nsIMdbEnv* mev, // context
- mdb_column* outColumn, // column for this particular cell
- mdb_pos* outPos, // position of cell in row sequence
- nsIMdbCell** acqCell)
-{
- nsresult outErr = NS_OK;
- nsIMdbCell* outCell = 0;
- mdb_pos pos = 0;
- mdb_column col = 0;
- morkRow* row = 0;
- morkEnv* ev = morkEnv::FromMdbEnv(mev);
- if ( ev )
- {
- pos = mCursor_Pos;
- morkCell* cell = row->CellAt(ev, pos);
- if ( cell )
- {
- col = cell->GetColumn();
- outCell = row->AcquireCellHandle(ev, cell, col, pos);
- }
- outErr = ev->AsErr();
- }
- if ( acqCell )
- *acqCell = outCell;
- if ( outPos )
- *outPos = pos;
- if ( outColumn )
- *outColumn = col;
-
- return outErr;
-}
-// } ----- end cell creation methods -----
-
// { ----- begin cell seeking methods -----
NS_IMETHODIMP
morkRowCellCursor::SeekCell( // same as SetRow() followed by MakeCell()
diff --git a/db/mork/src/morkRowCellCursor.h b/db/mork/src/morkRowCellCursor.h
index c9d95a2f24..76ad77891c 100644
--- a/db/mork/src/morkRowCellCursor.h
+++ b/db/mork/src/morkRowCellCursor.h
@@ -59,14 +59,6 @@ public: // morkRowCellCursor construction & destruction
NS_IMETHOD GetRow(nsIMdbEnv* ev, nsIMdbRow** acqRow) override;
// } ----- end attribute methods -----
- // { ----- begin cell creation methods -----
- NS_IMETHOD MakeCell( // get cell at current pos in the row
- nsIMdbEnv* ev, // context
- mdb_column* outColumn, // column for this particular cell
- mdb_pos* outPos, // position of cell in row sequence
- nsIMdbCell** acqCell) override; // the cell at inPos
- // } ----- end cell creation methods -----
-
// { ----- begin cell seeking methods -----
NS_IMETHOD SeekCell( // same as SetRow() followed by MakeCell()
nsIMdbEnv* ev, // context
diff --git a/mailnews/import/outlook/src/MapiApi.cpp b/mailnews/import/outlook/src/MapiApi.cpp
index d6a1597549..d23b44dd1f 100644
--- a/mailnews/import/outlook/src/MapiApi.cpp
+++ b/mailnews/import/outlook/src/MapiApi.cpp
@@ -887,7 +887,7 @@ BOOL CMapiApi::IterateStores(CMapiFolderList& stores)
if (!m_lpSession) {
MAPI_TRACE0("IterateStores called before session is open\n");
- m_lastError = -1;
+ m_lastError = E_UNEXPECTED;
return FALSE;
}
@@ -1124,7 +1124,7 @@ CMsgStore * CMapiApi::FindMessageStore(ULONG cbEid, LPENTRYID lpEid)
{
if (!m_lpSession) {
MAPI_TRACE0("FindMessageStore called before session is open\n");
- m_lastError = -1;
+ m_lastError = E_UNEXPECTED;
return NULL;
}
@@ -1409,9 +1409,6 @@ void CMapiApi::ReportLongProp(const char *pTag, LPSPropValue pVal)
else if (pVal && (PROP_TYPE(pVal->ulPropTag) == PT_ERROR)) {
MAPI_TRACE1("%s {Error retrieving property}\n", pTag);
}
- else if (pVal && (PROP_TYPE(pVal->ulPropTag) == PT_ERROR)) {
- MAPI_TRACE1("%s {Error retrieving property}\n", pTag);
- }
else {
MAPI_TRACE1("%s invalid value, expecting long\n", pTag);
}
@@ -1843,7 +1840,7 @@ BOOL CMapiFolderContents::SetUpIter(void)
}
if (ulObjType != MAPI_FOLDER) {
- m_lastError = -1;
+ m_lastError = E_UNEXPECTED;
MAPI_TRACE0("CMapiFolderContents - bad object type, not a folder.\n");
return FALSE;
}
diff --git a/mailnews/mime/emitters/nsEmitterUtils.cpp b/mailnews/mime/emitters/nsEmitterUtils.cpp
index 551bf5b318..2867596ab1 100644
--- a/mailnews/mime/emitters/nsEmitterUtils.cpp
+++ b/mailnews/mime/emitters/nsEmitterUtils.cpp
@@ -54,7 +54,6 @@ EmitThisHeaderForPrefSetting(int32_t dispType, const char *header)
(!strcmp(header, HEADER_FOLLOWUP_TO)) ||
(!strcmp(header, HEADER_CC)) ||
(!strcmp(header, HEADER_ORGANIZATION)) ||
- (!strcmp(header, HEADER_REPLY_TO)) ||
(!strcmp(header, HEADER_BCC))
)
return true;
diff --git a/mailnews/mime/src/mimemsig.cpp b/mailnews/mime/src/mimemsig.cpp
index bb34607cba..5b4f8e556a 100644
--- a/mailnews/mime/src/mimemsig.cpp
+++ b/mailnews/mime/src/mimemsig.cpp
@@ -599,7 +599,6 @@ MimeMultipartSigned_emit_child (MimeObject *obj)
obj->options->headers != MimeHeadersCitation &&
obj->options->write_html_p &&
obj->options->output_fn &&
- obj->options->headers != MimeHeadersCitation &&
sig->crypto_closure)
{
// Calling crypto_generate_html may trigger wanted side effects,