libs/kworkspace/kdisplaymanager.cpp mess
Aaron J. Seigo
aseigo at kde.org
Fri May 3 15:01:07 BST 2013
On Thursday, May 2, 2013 13:12:55 Martin Briza wrote:
> clear enough due to my English): this doesn't break anything nor introduce
> any kind of regression, the functionality stays the same.
Well, code speaks louder than words in any language. I would suggest creating a branch to do
this work in and start writing code. We can look at it as you progress and figure things like this out
as we go.
.. and I just read your follow up email saying you are doing exactly that. Awesome. :)
> Yes, I totally agree with the abstract base class idea, though I still
> think having a private subclass is a bit messy practice. Yet once again,
It is less messy than having a publicly exposed factory with every user of the library having to deal
with that implementation detail. "Messy" library internals are often necessary to make an elegant
API. It also would mean we don't have to port any code that currently uses this API which is
always a bonus: less work, fewer regressions.
--
Aaron J. Seigo
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130503/8f238a75/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130503/8f238a75/attachment.sig>
More information about the kde-core-devel
mailing list