Bug 1584473 - Fix several issues

* Fix undefined HRESULT codes in class CMapiApi in MapiApi.cpp.
* Fix doubled up sub-expressions in MIME and MAPI.
* Remove broken/unused morkRowCellCursor::MakeCell()

Tag #1273
This commit is contained in:
Matt A. Tobin 2019-11-11 02:08:59 -05:00 • committed by Roy Tam
commit e7cf7528e2
6 changed files with 3 additions and 60 deletions

View file

@ -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;
}

View file

@ -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;

View file

@ -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,