[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-pim
Subject:    Re: [Kde-pim] Fwd:  Fwd: [patch] kdepimlibs - gpgme++
From:       Marc Mutz <marc () klaralvdalens-datakonsult ! se>
Date:       2007-09-27 7:41:31
Message-ID: 200709270741.35624.marc () klaralvdalens-datakonsult ! se
[Download RAW message or body]

On Wednesday 26 September 2007 17:37, Jarosław Staniek wrote:
> Marc Mutz said the following, On 2007-09-26 16:32:
> > On Wednesday September 26 2007 16:23, Till Adam forwarded:
> >> A fix for typedefs under msvc7.1, for review.
> >>
> >> -    gpgme_key_t key
> >> +    struct _gpgme_key* key
> >
> > This is not acceptable. What's the issue this is trying to solve?
>
> Portability. Here's example:
> ---------
> [  0%] Building CXX object gpgme++/CMakeFiles/gpgmepp.dir/context.obj
> context.cpp
> F:\kde4\trunk\KDE\kdepimlibs\gpgme++\context.cpp(483) : error C2275:
> 'gpgme_key_t' : illegal use of this type as an expr
> ession

Note the use of gpgme_key_t in signingKey() just a few lines above is not 
flagged.

This thread
  http://lists.cs.uiuc.edu/pipermail/llvmdev/2004-September/001942.html
has a less intrusive solution. Please try that one.

> Optionally we could just include gpgme.h.

Not an option. We don't want to depend on gpgme.h through public headers.

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]