IconTasks taskmanager changes
Aaron J. Seigo
aseigo at kde.org
Thu Oct 27 12:29:58 UTC 2011
On Thursday, October 27, 2011 14:10:50 Craig Drummond wrote:
> > yes, it's an edge case for certain. would be nice if it works, all the
> > same.
>
> But, is it worth adding work-arounds for such an edge case? Why bloat the
> code with something that is unlikely to occur? At least I dont see it as a
because ime:
* covering edge cases often opens the door to fixing other similar cases later
on
* it helps prevent code from becoming overly specified and brittle by only
doing the "main" things well
* it gives people who try such things (out of innocence usually :) a really
good feeling that it actually works in all cases (as far as they know)
counter points to this are:
* it can lead the code to becoming spaghetti or a series of workarounds that
make it unmaintainable
if that condition is met, ime, you can mostly forget about the positives. so
it does become a matter of judgement. in this case, i don't think it would be
a big addition, would be applicable to other similar cases and would make sure
the code itself was reasonably generic.
--
Aaron J. Seigo
humru othro a kohnu se
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA EE75 D6B7 2EB1 A7F1 DB43
KDE core developer sponsored by Qt Development Frameworks
-------------- 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/plasma-devel/attachments/20111027/7cf4fb0f/attachment.sig>
More information about the Plasma-devel
mailing list