[prev in list] [next in list] [prev in thread] [next in thread]
List: koffice-devel
Subject: Re: Compiling on KDE 3.3.1
From: Nicolas Goutte <nicolasg () snafu ! de>
Date: 2005-01-22 15:16:16
Message-ID: 200501221616.16105.nicolasg () snafu ! de
[Download RAW message or body]
On Friday 21 January 2005 17:52, Laurent Montel wrote:
> On Friday 21 January 2005 17:03, David Faure wrote:
> > On Friday 21 January 2005 16:35, Nicolas Goutte wrote:
> > > Or perhaps use -fvisibility only if KDE 3.3.2 is found. (I do not know
> > > if it is possible to do so.)
> >
> > That's not the point. The problem is the FOO_EXPORT macros added
> > everywhere.
> >
> > [Note also that 3.3.2 still had a broken kdemacros.h. Only 3.4 has the
> > correct stuff,]
> >
> > Oh. I see. kdepim used KDE_EXPORT, which is fine for gcc, but isn't
> > enough for MSVC++. It should be a per-lib macro, for MSVC++.
> >
> > So while we're at it, we could do it right:
> > in koffice_export.h we would do like
> > #if KDE_IS_VERSION(3,3,90)
> > #define KOFFICECORE_EXPORT KDE_EXPORT
> > .. (same thing with KOFFICEUI_EXPORT etc.)
> > #else
> > #define KOFFICECORE_EXPORT
> > .. (same thing with KOFFICEUI_EXPORT etc.)
> > #endif
>
> it will be a very good idea.
>
> a kofficemacros.h :
> #ifndef KOFFICEMACROS_H
> #define KOFFICEMACROS_H
>
> #include <kdeversion.h>
>
> #if KDE_IS_VERSION(3,3,90)
>
> #include <kdemacros.h>
Better always #include <kdemacros.h> as it defines for example KDE_DEPRECATED
too..
>
(...)
Have a nice day!
_______________________________________________
koffice-devel mailing list
koffice-devel@kde.org
https://mail.kde.org/mailman/listinfo/koffice-devel
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic