path: root/TODO

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.

- 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.
	[email protected]

- 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).

- 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.