KDE_VERSION

[Harri Porten]

> #if KDE_VERSION > KDE_IS_VERSION(3,2,0)

What about putting this into the header as well? This doesn't look very 
self-explaining.

Fully agreed. Is the "IS" supposed to stand for the English "is" ? Then
above expression is really hard to understand logically ?

We gather some suggestions. How about:

#if KDE_VERSION_GT(3, 2, 0)    // and KDE_VERSION_LT

or

#if KDE_VERSION_REQUIRE(3, 2, 0)  // the glibc way

That way KDE_VERSION could be hidden completely. If that's not desired I
still suggest to rename KDE_IS_VERSION to something like KDE_MAKE_VERSION
or so.

Harri.