Anton S. wrote:
> I just downloaded recent version from SVN and found a strange define
> in OverbyteIcsDefs.inc {$IFDEF CLR}
>  {$DEFINE BOMB}
> {$ELSE}
>  {$IFNDEF COMPILER7_UP}
>    {$DEFINE BOMB}
>  {$ENDIF}
> {$ENDIF}
> It seems that it shouldn't really be CLR but old versions of BCB or
> something else. 

It's not for .NET however for Delphi 7+ and BCB 2006+, what is the
problem?

> 
> And the second thing, I faced with Range check errors inside ICS
> code. After some investigations I've found the root of evil:
> _PostMessage in TCustomWSocket.Send takes wParam as Integer while
> FHSocket it is called with has unsigned type Cardinal. Casting
> FHSocket to Integer resolved the issue. 

FHSocket actually is of type TSocket = u_int = Integer defined in 
Winsock.pas, WPARAM and LPARAM are of type Integer defined in 
Windows.pas, there is no cast required. Do you use Delphi?

> And consider using WPARAM/LPARAM types for casting instead of
> general integer ones for higher compatibility.

Agreed.

> Moreover, after I've
> learned some thing regarding x64 I accepted the habit of
> sending/posting pointers/handles ONLY in LParam which is what it
> intended for. So it'd be nice to see it in ICS.         

Currently both WPARAM and LPARAM are 32-bit and will be 64-bit 
in x64, no problem IMO. 

-- 
Arno Garrels
--
To unsubscribe or change your settings for TWSocket mailing list
please goto http://lists.elists.org/cgi-bin/mailman/listinfo/twsocket
Visit our website at http://www.overbyte.be

Reply via email to