[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-pim
Subject: Re: [Kde-pim] [patch] kdepim fixes for msvc 2k3+
From: Marc Mutz <marc () klaralvdalens-datakonsult ! se>
Date: 2007-12-08 8:30:17
Message-ID: 200712081032.02078.marc () klaralvdalens-datakonsult ! se
[Download RAW message or body]
On Saturday 08 December 2007 01:42, Jarosław Staniek wrote:
> - mResult = GpgME::KeyListResult( 0, err );
> + mResult = GpgME::KeyListResult( 0, err.toErrorCode() );
> return err;
Please make GpgME::*Result take a GpgME::Error instead, to minimize the
changes to external code.
> Index: libkleo/backends/chiasmus/chiasmusjob.cpp
> ===================================================================
> --- libkleo/backends/chiasmus/chiasmusjob.cpp (revision 746135)
> +++ libkleo/backends/chiasmus/chiasmusjob.cpp (working copy)
> @@ -103,7 +103,8 @@
>
> LaterDeleter d( this );
>
> - if ( const GpgME::Error err = setup() )
> + const GpgME::Error err = setup();
> + if ( err )
> return mError = err;
Veto. There must be a better way. 2k3 is pretty conforming, afair, I'm very
surprised it doesn't manage to compile these declarations. Please find the
real reason for the compiler breakage, instead of this ugly band-aid.
> +#undef max
There's a well-know fix for this, it's to define NOMINMAX, documented e.g.
here:
http://support.microsoft.com/kb/143208
I suggest you add that globally to the build system, instead of fixing it
locally.
Thanks,
Marc
--
Marc Mutz -- marc@klaralvdalens-datakonsult.se, mutz@kde.org
Klarälvdalens Datakonsult AB, Platform-independent software solutions
_______________________________________________
KDE PIM mailing list kde-pim@kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic