kdelibs/win32 : kdecore patch

Jarosław Staniek js at iidea.pl
Wed Sep 8 08:27:55 BST 2004


Harri Porten wrote:

> I also think that those #ifdefs are sometimes placed at strategically
> suboptimal points. Instead of disabling all those places *using* a class
> it's better to make the changes to the class itself. Either by writing a
> new (possibly dummy) class or putting #ifdefs into the implementation of
> the original class.

I must admit there are few completely ifdef'd (using Q_WS_WIN, W_OS_UNIX etc) 
methods especially in kdecore now. Most of these are waiting for such wrapper 
classes or just wrapped methods, implemented in *_win.cpp files (as Qt does), 
with volunteers' help :)

-- 
regards / pozdrawiam,
  Jaroslaw Staniek / OpenOffice Polska
  Kexi Project: http://www.kexi-project.org, http://koffice.kde.org/kexi
  QT-KDE Wrapper Project: http://iidea.pl/~js/qkw




More information about the kde-core-devel mailing list