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
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel
