[Lazarus] THelpEvent declaration discussion

[Marco van de Voort]

On Fri, Nov 06, 2009 at 12:18:15PM +0200, Graeme Geldenhuys wrote:
> > But what I do not understand is why you need this identifier to change at
> > all? Let it be there, all nice and Delphi compatible, and define your own
> > for your own systems.
> 
> As I said above. FPC is for multiple platforms, not just Windows. Many
> other features have changed in FPC for example that reason - even if
> that means breaking Delphi compatibility.
> 
> TComponent is part of FPC. So is THelpContent and THelpEvent, THelpType
> etc... So why reinvent the wheel (or declare yet more custom type), when
> we should be able to use what is available in FPC itself - and built on
> top of that. Irrespective of what GUI toolkit or platform we use.

This is not the same. Other parts are changed when it is not possible
otherwise. In other places, the default identifiers are reserved for Delphi
compatibility, and 3rd party alternatives define their own.

> What makes my point in more appropriate is that Lazarus doesn't even use
> the THelpEvent as defined in Classes unit. They implemented there own
> type in the Forms.pp unit, so no, the change I suggest will not even
> break Lazarus code.

I don't see anything appropriate there, and no need to change this at all