On Sat, 23 Jan 2010 01:32:05 +1000 Alexander Klenin <[email protected]> wrote:
> On Sat, Jan 23, 2010 at 00:59, Paul Ishenin <[email protected]> wrote: > > 22.01.2010 21:07, Alexander Klenin wrote: > >> At the current stage, I simply re-export TPropertyEditHook > >> from propedits unit. > >> Theoretically, after updating all dependent units > >> this re-export might be removed, but there is no pressure -- > >> it is only a single line and can be postponed indefinitely. > > > > Refactoring should not be the final goal. Refactoring usually used when > > something fit bad into the existing design. > > That is one possible reason. > The other reason (which was the actual reason I undertook the refactoring) > is that, at least for me, there is a certain threshold of ugliness > after which I am just > unable to unravel the current design and understand it enough > to make a needed change, even if the design itself can accommodate it. > This property is often called 'unmaintainability'. > Of course, the threshold is different for different people, but I consider > 6000 LOC in a singe unit to be far beyond any reasonable level. General speaking: Readability alone is a bad reason to break compatibility. Readability can be improved by comments, class-trees, examples, tutorials and include files. >[...] Mattias -- _______________________________________________ Lazarus mailing list [email protected] http://lists.lazarus.freepascal.org/mailman/listinfo/lazarus
