Ticket #61 (new enhancement)

Opened 20 months ago

Last modified 6 months ago

Code cleanup and improvement tasks

Reported by: diederik Owned by: diederik
Priority: someday Milestone: open
Component: Other Version: 1.5-pre2
Keywords: Cc:

Description (last modified by diederik) (diff)

These are some tasks to cleanup the code:

  • Create a WinkArchive class for the code in ChatWindow::showWink() (also see ticket #20).
  • Use kdWarning() instead of kdDebug() << "WARNING", for warning messages.
  • Use if( KMESS_NULL(..) ) checks instead of current if(.. != 0) { kdDebug() .. }
  • Remove the dependency on QWidget for data-classes.
  • Move utility classes to a separate folder.

And some to improve the current API:

  • Include contact meta-data in receivedMessage() call of ChatWindow::showWink().
  • Add Application::get..TransferAbortMessage().
  • Implement Application::attachToChat(..) to improve aborting (open for discussion).
  • Rename "Picture Transfer" to "Msn Object Transfer" everywhere.

Improve performance of restyling the chat window: combine xml to one string.

Attachments

kdwarnings.patch (3.8 kB) - added by amroth 13 months ago.
Had this one lying around for a long time. Good thing it still applies without further work :D

Change History

Changed 15 months ago by amroth

  • milestone set to Someday

Changed 13 months ago by amroth

Had this one lying around for a long time. Good thing it still applies without further work :D

Changed 13 months ago by diederik

  • description modified (diff)

added note for picturetransferp2p.

Changed 12 months ago by diederik

  • description modified (diff)

removed need for TcpConnectionBase, it's no longer needed since the KDE 4 port because QSocket does what we need.

Changed 6 months ago by valerio

Do we still need this ticket to be open? Maybe it needs to be rewritten?

Note: See TracTickets for help on using tickets.