[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-devel
Subject: Re: [patch] request for review - apidox for kdelibs/kdeui (long)
From: Brad Hards <bhards () bigpond ! net ! au>
Date: 2003-09-09 9:52:07
[Download RAW message or body]
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On Tue, 09 Sep 2003 11:20 am, Nicolas Goutte wrote:
> Sorry to having made a confusion! It is the other way round. Of course, you
> define KDE_NO_COMPAT in a program only when you do *not* want
> compatibility. You make sure that it is not defined if you want
> compatibility to the previous major version.
OK, so #define KDE_NO_COMPAT only ever removes compatibility code.
In that case, wouldn't it make more sense not to define KDE_NO_COMPAT when
building the documentation:
Index: Doxyfile.global
===================================================================
RCS file: /home/kde/kde-common/admin/Doxyfile.global,v
retrieving revision 1.17
diff -u -4 -p -r1.17 Doxyfile.global
- --- Doxyfile.global 21 Aug 2003 09:46:11 -0000 1.17
+++ Doxyfile.global 9 Sep 2003 09:33:37 -0000
@@ -875,9 +875,9 @@ INCLUDE_FILE_PATTERNS =
# gcc). The argument of the tag is a list of macros of the form: name
# or name=definition (no spaces). If the definition and the = are
# omitted =1 is assumed.
- -PREDEFINED = QT_VERSION=320 __cplusplus KDE_NO_COMPAT
+PREDEFINED = QT_VERSION=320 __cplusplus
# If the MACRO_EXPANSION and EXPAND_ONLY_PREDEF tags are set to YES then
# this tag can be used to specify a list of macro names that should be
expanded.
# The macro definition that is found in the sources will be used.
In any case, we should document @deprecate'd API elements with at least the
replacement code.
Brad
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.0.7 (GNU/Linux)
iD8DBQE/XaLIW6pHgIdAuOMRAo4QAJ9/LPKNooKY9GJ6hrWIQxcqsIZ/yQCfXZ4L
5hf32GGDoJRZ1hSrDOX6VSc=
=+89G
-----END PGP SIGNATURE-----
>> Visit http://mail.kde.org/mailman/listinfo/kde-devel#unsub to unsubscribe <<
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic