Re: [Hwgui-developers] HwGUI - the continuation

[Alexander S.K. <al...@be...>]

Alex Strickland writes:
> On 2012/12/13 01:34 PM, Alexander S.Kresin wrote:
>>    Yet another typical things, which must be corrected:
>>
> I was not able to understand why this kind of code was necessary:
> 
>              ::oparent:lSuspendMsgsHandling := .T.

   As I see from the Changelog, it was introduced by Luis Fernando Basso:

2008-09-01 21:00 UTC+0100 Maurizio la Cecilia <m.l...@gm...>
   This commit contains all the changes made by Luis Fernando Basso
...
     * added lSuspendMsgsHandling property allowing centralized
       control of event handling

It would be nice if Luis ( can I call you that ? ) tell us in details 
about this.

> 
> It seems to be in every class.
> 
> This kind of code in hbrowse.prg is horrible:
> 
>        ::internal[ 1 ] := SetBit( ::internal[ 1 ], 1, 0 )

   This is my old code :) and, agree, it isn't good.

> 
> and this in hedit.prg :
> 
>              ELSEIF wParam == 39     // KeyRight
>                 ::lFocu := .F.
>                 IF ! IsCtrlShift()
>                    ::lFirst := .F.
>                    RETURN ::KeyRight()
>                 ENDIF
>              ELSEIF wParam == 37     // KeyLeft
> 
> Some people seem to think #define is evil.
> 

   Suggest your version :)

Regards, Alexander.