KConfigBase & TODO

[Thomas Braxton]

The TODO file has this entry
- KConfigBase: 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.

I was wondering if this patch is correct and if so should I remove the item 
from the TODO file or just mark it as done when I commit?
-------------- next part --------------
A non-text attachment was scrubbed...
Name: kconfigbase.diff
Type: text/x-diff
Size: 11846 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20051213/e2d674e2/attachment.diff>