This file is meant to be a place to track of medium to large-scale API changes that we would like to make for the next binary incompatible release. - Check for forked classes in kde pim and other modules - There is no reason why TDEConfigBase should inherit from QObject, get rid of that. - Change all FooPrivate *d; -> Private * const d; and place initialization in the constructor (for classes that would benefit from this). To help catch silly mistakes since d should never change. Also consider changing to use KStaticDeleter to help prevent mistakes where developers forget to delete the pointer. Maybe make use of Qt4 helper macros? - Move all utility functions away from TDEApplication. TBD: Make TDEApplication a very thin wrapper around QApplication. Ideally, TDEApplication should go away completely. The kapp pointer references everywhere must die. Find a way to make that possible - Remove all KDE_NO_COMPAT from the KDE2 -> KDE3 port before adding new ones for KDE4. - Remove the smart-to-dumb-pointer conversion operator from TDESharedPtr and replace it with a T* Get() member function. An implict conversion is far too dangerous. See: http://groups.google.com/groups?selm=4ognb0%243cd%40linda.teleport.com&oe=UTF-8&output=gplain Also free the client from the requirement to inherit from TDEShared. - Remove all TDESocket, TDEServerSocket, KSocks and KExtendedSocket references throughout KDE in favor of KNetwork's socket implementations. Also, revise the namespace usage. thiago.macieira@kdemail.net - Remove all methods marked 'deprecated' in KDE 3.x. - Clean up the uses of virtual_hook, use normal virtuals instead (e.g. KZip/KArchive) - Make it possible to create a KSystemTray without a TDEMainWindow and instead lazily create the main window on demand. - Make KSystemTray's Quit option in the context menu call kapp->quit() instead of closing all windows, so apps that save their window state on exit don't need workarounds to detect a window close caused by the tray rather than the user. - Make KSystemTray add its own reference to KApp, so tray apps that adhere to the style guide can safely close their main window without having to worry about the app quitting instead. - Make some long-lasting operations threaded. There are some operations in KDE that take too long on at least ~500MHz hardware to be done using event loops. Examples are the use of KLibLoader to dlopen plugins and KRun to fire up other processes. Konqueror and KMail tend to stall occasionally too, but that's harder to refactor because the bulk of the work there is X11-related. - Make it possible to use SSL-ed sockets outside TDEIO. KSSL isn't very suitable for reuse, and the certificate dialogs are not even available in public API at all, leading e.g. to Kopete IRC duplicating a lot of stuff and all KDE Jabber clients based on libpsi (PSI, Light Hawk and Kopete) using homebrewn solutions. [How does this have to do with changing TDEIO? It's just a request to have that code in a different form....] - Manually, properly clean up the includes in all of the headers. Remove unnecessary ones and change pointers to use class foo; decorations and move the headers into the cpp files. When finished fixkdeincludes shouldn't find anything (but don't rely on fixkdeinclude, it should be done manually first!) - Use of the C++ keyword "explicit". Make all ctors explicit which can be used with one parameter to minimize wrong use of the class (i.e. have a look at Dirk's work on KURL). Another candidate: KService( KDesktopFile *config ). KGuiItem has one of these dangerous ctors, too. KUser has a whole bunch of them. - Solve need for casting when using bitfield enums (e.g. TDEProcess::Communication), if Simon's solution works. - Merge tdelibs/tdecore/kuser.h and tdeadmin/kuser/kuser.h. - Simplify memory management with KParts. The part and the widget delete each other, which in fact makes it more complex to handle. The part owns the widget so it should delete it, but not the other way round. - KParts: remove plugin loading from setInstance, add an explicit loadPlugins() call in all parts that should load plugins (at the end of the derived-part ctor). - KParts: come up with better names for ReadOnlyPart and ReadWritePart, ReadOnly is a minimum requirement, but this does not become clear until you dig deeper into the class. With QTextEdit and QTextBrowser for instance, the inheritance is the other way around. E.g.: ReaderPart? ViewerPart? - Other ideas in this thread: http://lists.kde.org/?l=kde-core-devel&m=107608067302571&w=2 (Contributors to that thread, please add specific items here.) - TDEConfigBase: For all the read*Entry methods that take a QFont *, QPoint *, QSize *, etc. for the default value, add overloads that use a const reference, so that one can write resize( config->readSizeEntry( "Bleh", size() ); for example. Constructing a null QFont/QPoint/QSize is cheap enough to justify a consistent and usable API (IMHO) . Keep source compatibility. - Sort out reference counting of applications and TDEMainWindows: Add a setRefCountEnabled() function to TDEApplication; deref()ing the last reference should only quit the application if reference counts are enabled, and the reference count should be disabled by default so apps that don't use a TDEMainWindow mostly continue to work. Make the TDEMainWindow constructor call setRefCountEnabled(true), and make each TDEMainWindow hold its own reference to the application. The TDEMainWindow should drop its app reference on hide, and regain it on show. Make KSysTray also hold a reference to the app. Then apps that use more than one main window and the tray (like KSIrc and Kopete) will quit at the right time without nasty hacks. - Change classes that store bool data using strings ("true"/"false") to use bool calls (readBoolEntry()). Although not technicaly a binary incompatiblity... just something to bring up for possible change (for the better) Use TDEConfigXT instead! /FransE - Is it really necessary to bother people using the KCM APIs with withFallback? I have never heard of it failing and when would you not like to have that fallback? I think a default of 'true' would be fine(and remove the argument in the APIs). - Do we need to have factory arguments in the KCM API? (who would use it anyway?) - TDECModuleInfo should use d-pointer. Perhaps there is more classes. - The config options in kdeglobals needs to be cleaned up a little. Perhaps I'm being picky.. * They should be named properly, for example "enableIcons" instead of "EnableIcons". Or atleast settle on one thing. * AFAICT it would be the right thing if KWin had its own rc file instead of having its "WM" section in kdeglobals. * Shouldn't KSpell have its own rc file instead of kdeglobals? * Should we allow spaces in option names? * Some option names are too short for being understandable. fixed->fixedFont; font->systemWideFont * Perhaps an own font section? - kpassdlg.h needs to be renamed to kpassworddialog.h - consistent with other classes(it won't break many apps). The class needs a facelift - see the comments in kpassdlg.{h,cpp} - TDEProcess needs a major facelift. Not all of these items need to be BIC, but not needing to worry about legacy will certainly allow an overall cleaner solution. - the internals need to be encapsulated properly. many of the virtuals shouldn't be virtual; lots of protected members should be accessible only via accessors. the current model is theoretically super-flexible, but in fact it is only super-fragile. - clear up pipe() vs. socketpair() usage. - move the draining loop from commClose() to wait() - add additional "channels" beyond std*, with the built-in capability of passing the fd on the command line - redirections, including piping multiple kprocesses together. this should make most cases of reimplementing commSetupDone{P,C} superflous. - possibly create KForkProcess that covers most/all of the remaining commSetupDoneC reimplementations. - do something about the NoRead comm mode - add setComm(); remove comm arg from start(). as a side effect, this will allow for using writeStdin() in Block mode. - merge KProcIO into TDEProcess; add setBuffered() for separate channels. - use QByteArray for writeStdin(). better than the buf+len arg, as it is ref-counted. - To discuss: Migrate the about dialog to use qhtml or tdehtml rather then the current setup of all of the widgets and sub widgets. Might be a lot simpler, require a lot less code and be much more flexible for additions/enhancments. Sidenote: currently the about information seems to be duplicated. - Make API use US English, (eg. KCatalogue -> KCatalog) - It would be nice if some insan^H^H^H brave soul had a look at the TDEAccel/TDEShortcut/KKeyWhatever classes and cleaned it up. - KKeyDialog needs to be improved. It should simplify using KKeyChooser by not having to instantiate KDialogBase without restricting the possibilities of KKeyChooser (for instance, being able to configure global and application accels in one dialog). If there is only little time, simply expose the KKeyChooser object instead of making it private and inaccessible. - Fix the APIs taking Matthias Ettrich's hints into account, most importantly fix the massive amount of bool-usage in the CTORs (see KDialogBase, for instance) - Get rid of KXMLClient::conserveMemory() and the conserveMemory argument in createGUI. It created too many problems in the past. Not worth the trouble. - Split KProtocolInfo into KProtocolInfoBase in tdecore (with almost everything) and KProtocolInfo in tdeio (with the methods taking a KURL, which need KProtocolManager) - Rename KDirNotify_stub and update API (no need for '*' arguments anymore) - TDEWallet::Wallet::* functions - remove the default = 0 argument for mainwindow for the dialog - Make libtdewalletclient part of libtdeio and get rid of libtdewalletclient. - Get rid of libtdesu dependency in libtdeio, get rid of SessionData::AuthData*, get rid of SlaveInterface::authorizationKey and SlaveInterface::delAuthorization - Make functions in KHelpMenu virtual so they can be overridden (ex: to provide an alternative help->contents action) - Replace KPasswordEdit with KLineEdit and adjust KPasswordDialog accordingly. - Merge KPIM::ClickLineEdit with KLineEdit - Merge all K*Label classes into one that offers all the features such as ellipsis (this time for center, middle and right in order to allow for sqeezing the text), links and selectable text with a consistant API. - Make sure all GUI classes adopt well to palette changes. - Rework KPanelExtension and KPanelApplet to have a sane set of global position enums, make the geometry handling less spaghetti like and extend the API so requests for things like re-reading the config of a KPanelExtension can be done by its parent. - Fix KURLRequester API to use KURL for urls instead of QString to make clear that we work with URLs and not with paths. - Dump KPixmapIO class. QPixmap with qt-copy patches #0005 and #0007 can perform just as well, KPixmapIO can't dither, and it generally doesn't make much sense to have this class. - TDEListView: merge setDropHighlighter and setDropVisualizer, add a setDnDMode to choose between "dnd to move items" (e.g. keditbookmarks) and "dnd onto items" (e.g. kmail). Unless Qt4 does it all :) - Move TDEIO::findDeviceMountPoint, findPathMoundPoint, probably_slow_mounted, and testFileSystemFlag to KMountPoint, to merge that code. - Refactor the tdeio/bookmarks code so we don't need to use the static d-pointer trick anymore. - Rename the parameters to KDEDesktopMimeType::pixmap so that they use meaningful names rather than just a, b, c, and d. Probably not something that has to wait for KDE 4, but seems like a good idea to have here for later doing. - Move KRichTextLabel into tdeui if still required with Qt 4. - Add the concept of a session to TDEIO, in particular for TDEHTML so that it can have all of its jobs associated in some way (a unique key of sorts). Will make SSL much easier to implement and allow removal of many hacks, mostly involving metadata. - Make KLibLoader default to RTLD_GLOBAL and make necessary changes to make KDE namespace clean.