diff options
Diffstat (limited to 'src/core')
-rw-r--r-- | src/core/basetypes/MCAssert.c | 2 | ||||
-rw-r--r-- | src/core/basetypes/MCBaseTypes.h | 1 | ||||
-rw-r--r-- | src/core/basetypes/MCData.cpp | 2 | ||||
-rw-r--r-- | src/core/basetypes/MCHTMLCleaner.h | 4 | ||||
-rw-r--r-- | src/core/basetypes/MCLog.cpp | 8 | ||||
-rw-r--r-- | src/core/basetypes/MCString.cpp | 96 | ||||
-rw-r--r-- | src/core/basetypes/MCString.h | 3 | ||||
-rw-r--r-- | src/core/imap/MCIMAPMessage.cpp | 1 | ||||
-rwxr-xr-x | src/core/imap/MCIMAPSession.cpp | 1 | ||||
-rw-r--r-- | src/core/nntp/MCNNTPGroupInfo.cpp | 2 | ||||
-rw-r--r-- | src/core/renderer/MCHTMLRenderer.cpp | 25 | ||||
-rw-r--r-- | src/core/renderer/MCHTMLRendererCallback.cpp | 4 | ||||
-rw-r--r-- | src/core/renderer/MCHTMLRendererCallback.h | 2 |
13 files changed, 143 insertions, 8 deletions
diff --git a/src/core/basetypes/MCAssert.c b/src/core/basetypes/MCAssert.c index ce9b5703..564f1648 100644 --- a/src/core/basetypes/MCAssert.c +++ b/src/core/basetypes/MCAssert.c @@ -9,6 +9,6 @@ void MCAssertInternal(const char * filename, unsigned int line, int cond, const return; } - fprintf(stderr, "%s:%i: assert %s\n", filename, line, condString); + fprintf(stderr, "%s:%u: assert %s\n", filename, line, condString); abort(); } diff --git a/src/core/basetypes/MCBaseTypes.h b/src/core/basetypes/MCBaseTypes.h index ce3e1153..3088d4c2 100644 --- a/src/core/basetypes/MCBaseTypes.h +++ b/src/core/basetypes/MCBaseTypes.h @@ -26,5 +26,6 @@ #include <MailCore/MCICUTypes.h> #include <MailCore/MCIterator.h> #include <MailCore/MCConnectionLogger.h> +#include <MailCore/MCHTMLCleaner.h> #endif diff --git a/src/core/basetypes/MCData.cpp b/src/core/basetypes/MCData.cpp index d9f14efa..079344da 100644 --- a/src/core/basetypes/MCData.cpp +++ b/src/core/basetypes/MCData.cpp @@ -725,10 +725,10 @@ ErrorCode Data::writeToFile(String * filename) return ErrorFile; } size_t result = fwrite(bytes(), length(), 1, f); + fclose(f); if (result == 0) { return ErrorFile; } - fclose(f); return ErrorNone; } diff --git a/src/core/basetypes/MCHTMLCleaner.h b/src/core/basetypes/MCHTMLCleaner.h index 33e65767..eccf88c9 100644 --- a/src/core/basetypes/MCHTMLCleaner.h +++ b/src/core/basetypes/MCHTMLCleaner.h @@ -13,6 +13,8 @@ #include <MailCore/MCString.h> #include <MailCore/MCUtils.h> +#ifdef __cplusplus + namespace mailcore { class MAILCORE_EXPORT HTMLCleaner { @@ -23,3 +25,5 @@ namespace mailcore { } #endif + +#endif diff --git a/src/core/basetypes/MCLog.cpp b/src/core/basetypes/MCLog.cpp index 952229ec..386ac422 100644 --- a/src/core/basetypes/MCLog.cpp +++ b/src/core/basetypes/MCLog.cpp @@ -53,9 +53,7 @@ static void logInternalv(FILE * file, return; while (1) { - const char * p = filename; - - p = strchr(filename, '/'); + const char * p = strchr(filename, '/'); if (p == NULL) { break; } @@ -80,7 +78,7 @@ static void logInternalv(FILE * file, #else if (0) { #endif - fprintf(file, "[%i:main] %s:%i: ", sPid, filename, line); + fprintf(file, "[%i:main] %s:%u: ", sPid, filename, line); } else { unsigned long threadValue; @@ -91,7 +89,7 @@ static void logInternalv(FILE * file, #else threadValue = (unsigned long) thread_id; #endif - fprintf(file, "[%i:%lx] %s:%i: ", sPid, threadValue, filename, line); + fprintf(file, "[%i:%lx] %s:%u: ", sPid, threadValue, filename, line); } vfprintf(file, format, argp); fprintf(file, "\n"); diff --git a/src/core/basetypes/MCString.cpp b/src/core/basetypes/MCString.cpp index 02c93200..7d7a740e 100644 --- a/src/core/basetypes/MCString.cpp +++ b/src/core/basetypes/MCString.cpp @@ -2515,6 +2515,102 @@ Data * String::decodedBase64Data() return result; } +static int hexValue(const char * code) { + int value = 0; + const char * pch = code; + for (;;) { + int digit = *pch++; + if (digit >= '0' && digit <= '9') { + value += digit - '0'; + } + else if (digit >= 'A' && digit <= 'F') { + value += digit - 'A' + 10; + } + else if (digit >= 'a' && digit <= 'f') { + value += digit - 'a' + 10; + } + else { + return -1; + } + if (pch == code + 2) { + return value; + } + value <<= 4; + } +} + +String * String::urlDecodedString() +{ + Data * sourceData = dataUsingEncoding(); + const char * source = sourceData->bytes(); + char * start = (char *) malloc(sourceData->length() + 1); + char * dest = start; + unsigned int i = 0; + while (i < sourceData->length()) { + switch (source[i]) { + case '%': + { + if (i + 2 < sourceData->length()) { + int value = hexValue(&source[i + 1]); + if (value >= 0) { + *(dest++) = value; + i += 3; + } + else { + *dest++ = '?'; + i ++; + } + } + else { + *dest++ = '?'; + i ++; + } + break; + } + default: + { + *dest++ = source[i]; + i ++; + break; + } + } + } + * dest = 0; + String * result = String::stringWithUTF8Characters(start); + free(start); + return result; +} + +static inline bool isValidUrlChar(char ch) { + return strchr("$&+,/:;=?@[]#!'()* ", ch) == NULL; +} + +String * String::urlEncodedString() +{ + const char * digits = "0123456789ABCDEF"; + Data * sourceData = dataUsingEncoding(); + const char * source = sourceData->bytes(); + char * start = (char *) malloc(sourceData->length() * 3 + 1); + char * dest = start; + unsigned int i = 0; + while (i < sourceData->length()) { + unsigned char ch = (unsigned char) source[i]; + if (isValidUrlChar(ch)) { + *dest++ = ch; + } else { + *dest++ = '%'; + *dest++ = digits[(ch >> 4) & 0x0F]; + *dest++ = digits[ ch & 0x0F]; + } + i ++; + } + *dest = 0; + String * result = String::stringWithUTF8Characters(dest); + free(start); + + return result; +} + HashMap * String::serializable() { HashMap * result = Object::serializable(); diff --git a/src/core/basetypes/MCString.h b/src/core/basetypes/MCString.h index cbe5538b..5ee2ad65 100644 --- a/src/core/basetypes/MCString.h +++ b/src/core/basetypes/MCString.h @@ -117,6 +117,9 @@ namespace mailcore { virtual Data * decodedBase64Data(); + virtual String * urlDecodedString(); + virtual String * urlEncodedString(); + public: // private static String * uniquedStringWithUTF8Characters(const char * UTF8Characters); diff --git a/src/core/imap/MCIMAPMessage.cpp b/src/core/imap/MCIMAPMessage.cpp index 2c8ef9b5..530d9075 100644 --- a/src/core/imap/MCIMAPMessage.cpp +++ b/src/core/imap/MCIMAPMessage.cpp @@ -26,6 +26,7 @@ void IMAPMessage::init() mModSeqValue = 0; mGmailThreadID = 0; mGmailMessageID = 0; + mSize = 0; } IMAPMessage::IMAPMessage() diff --git a/src/core/imap/MCIMAPSession.cpp b/src/core/imap/MCIMAPSession.cpp index 9fcf8397..d070b853 100755 --- a/src/core/imap/MCIMAPSession.cpp +++ b/src/core/imap/MCIMAPSession.cpp @@ -1578,7 +1578,6 @@ void IMAPSession::appendMessageWithCustomFlagsAndDate(String * folder, Data * me mProgressCallback = progressCallback; bodyProgress(0, messageData->length()); - flag_list = NULL; flag_list = flags_to_lep(flags); if (customFlags != NULL) { for (unsigned int i = 0 ; i < customFlags->count() ; i ++) { diff --git a/src/core/nntp/MCNNTPGroupInfo.cpp b/src/core/nntp/MCNNTPGroupInfo.cpp index 787967f3..8ebe08bc 100644 --- a/src/core/nntp/MCNNTPGroupInfo.cpp +++ b/src/core/nntp/MCNNTPGroupInfo.cpp @@ -13,10 +13,12 @@ using namespace mailcore; void NNTPGroupInfo::init() { mMessageCount = 0; + mName = NULL; } NNTPGroupInfo::NNTPGroupInfo() { + MC_SAFE_RELEASE(mName); init(); } diff --git a/src/core/renderer/MCHTMLRenderer.cpp b/src/core/renderer/MCHTMLRenderer.cpp index 60788d57..e2839610 100644 --- a/src/core/renderer/MCHTMLRenderer.cpp +++ b/src/core/renderer/MCHTMLRenderer.cpp @@ -218,6 +218,9 @@ static String * htmlForAbstractMessage(String * folder, AbstractMessage * messag context.pass = 1; context.firstAttachment = false; context.hasTextPart = false; + + htmlCallback->setMixedTextAndAttachmentsModeEnabled(context.hasMixedTextAndAttachments); + String * content = htmlForAbstractPart(mainPart, &context); if (content == NULL) return NULL; @@ -274,6 +277,28 @@ static String * htmlForAbstractSinglePart(AbstractPart * part, htmlRendererConte if (isTextPart(part, context)) { if (context->pass == 0) { if (context->state == RENDER_STATE_HAD_ATTACHMENT) { +#if 0 + if (part->className()->isEqual(MCSTR("mailcore::IMAPPart"))) { + if (mimeType->isEqual(MCSTR("text/plain"))) { + Data * data = context->dataCallback->dataForIMAPPart(context->folder, (IMAPPart *) part); + if (data != NULL) { + if (data->length() == 0) { + return NULL; + } + else if (data->length() == 2) { + if (strncmp(data->bytes(), "\r\n", 2) == 0) { + return NULL; + } + } + else if (data->length() == 1) { + if (strncmp(data->bytes(), "\n", 1) == 0) { + return NULL; + } + } + } + } + } +#endif context->state = RENDER_STATE_HAD_ATTACHMENT_THEN_TEXT; } return NULL; diff --git a/src/core/renderer/MCHTMLRendererCallback.cpp b/src/core/renderer/MCHTMLRendererCallback.cpp index fb492c9e..a3b14341 100644 --- a/src/core/renderer/MCHTMLRendererCallback.cpp +++ b/src/core/renderer/MCHTMLRendererCallback.cpp @@ -289,3 +289,7 @@ bool HTMLRendererTemplateCallback::shouldShowPart(AbstractPart * part) { return true; } + +void HTMLRendererTemplateCallback::setMixedTextAndAttachmentsModeEnabled(bool enabled) +{ +} diff --git a/src/core/renderer/MCHTMLRendererCallback.h b/src/core/renderer/MCHTMLRendererCallback.h index a12a3e45..4a4fc40f 100644 --- a/src/core/renderer/MCHTMLRendererCallback.h +++ b/src/core/renderer/MCHTMLRendererCallback.h @@ -35,6 +35,8 @@ namespace mailcore { HTMLRendererTemplateCallback(); virtual ~HTMLRendererTemplateCallback(); + virtual void setMixedTextAndAttachmentsModeEnabled(bool enabled); + virtual bool canPreviewPart(AbstractPart * part); virtual bool shouldShowPart(AbstractPart * part); |