[fpc-devel] [PATCH] patch for zipper.pp

[Andrew Haines]

Micha Nelissen wrote:

> 
> Do you mind if I say I think this is a very weird event handler?

No, of course not :)

> 
> What is the behaviour? How many times is it called? What does AClosing
> mean? How do I know I don't have a memory leak?
> 

Yes you are right of course. It is confusing.

> Why an event handler and not a property? A CustomStream property where
> non-nil means use this stream, if nil then default old behaviour e.g.
> 

As Michael said a property does not make much sense.

I just looked a the changes Michael made and it is very straight forward
 now :)

Regards,

Andrew