[Lazarus] IDEIntf: Extract PropHooks from PropEdits unit

[Mattias Gaertner]

On Sat, 23 Jan 2010 01:32:05 +1000
Alexander Klenin <klenin at gmail.com> wrote:

> On Sat, Jan 23, 2010 at 00:59, Paul Ishenin <webpirat at mail.ru> 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