zorba-coders team mailing list archive

[Gabriel Petrovay <gabriel.petrovay@xxxxxxxxxx>]

Gabriel Petrovay has proposed merging lp:~gabipetrovay/zorba/imap_warnings into lp:zorba/email-module.

Requested reviews:
  David Graf (davidagraf)

For more details, see:
https://code.launchpad.net/~gabipetrovay/zorba/imap_warnings/+merge/78718

Please quick look and fix this issue.


Thanks!
-- 
https://code.launchpad.net/~gabipetrovay/zorba/imap_warnings/+merge/78718
Your team Zorba Coders is subscribed to branch lp:zorba/email-module.

=== modified file 'src/com/zorba-xquery/www/modules/email/cclient/imap_client.cpp'
--- src/com/zorba-xquery/www/modules/email/cclient/imap_client.cpp	2011-10-05 13:00:46 +0000
+++ src/com/zorba-xquery/www/modules/email/cclient/imap_client.cpp	2011-10-08 16:04:23 +0000
@@ -23,7 +23,9 @@
 
 #include "email_exception.h"
 
+#ifndef WIN32
 #pragma GCC diagnostic ignored "-Wwrite-strings"
+#endif
 
 
 namespace zorba { namespace emailmodule {
@@ -69,7 +71,7 @@
       }
       else {
         sprintf (tmp, "MAIL");
-        res = smtp_mail(smtp_stream, tmp, aEnvelope, aBody);
+        res = smtp_mail(smtp_stream, tmp, aEnvelope, aBody) != 0;
         if (res) {
           out << "Ok.";
         } else {
@@ -931,4 +933,6 @@
 }
 
 
+#ifndef WIN32
 #pragma GCC diagnostic warning "-Wwrite-strings"
+#endif