[patch] Samples: change unsafe Integer(Pointer) typecasts to W/LPARAM

[Fr0sT.Brutal 900]

Ensures x64-safe typecasts in 6 units of old demos.

Note: Francois' name in unit headers got damaged by Git patch generator though the commit didn't contain that change. Must be reverted manually

0001-Samples-change-unsafe-Integer-Pointer-typecasts-to-W.patch

[Angus Robertson 577]

Your patches don't work on my working copy, I look for changes manually. 

 

The samples were never designed or tested to build on Win64, only the components, although we do test the actively supported samples periodically. 

 

Angus

 

[david_navigator 12]

20 minutes ago, Angus Robertson said:

Your patches don't work on my working copy, I look for changes manually. 

 

The samples were never designed or tested to build on Win64, only the components, although we do test the actively supported samples periodically. 

 

Angus

 

It's not just 64 bit

-            smtpQuit : PostMessage(Form1.Handle, WM_REMOVEOBJ, Integer(Sender), 0);
+            smtpQuit : PostMessage(Form1.Handle, WM_REMOVEOBJ, WPARAM(Sender), 0);

as discussed here,

 

[Fr0sT.Brutal 900]

1 hour ago, Angus Robertson said:

Your patches don't work on my working copy, I look for changes manually. 

 

The samples were never designed or tested to build on Win64, only the components, although we do test the actively supported samples periodically. 

 

Angus

 

That's weird. I based them on V8.66 - Part 10. Probably that encoding issue of Francois' name is the reason.

 

[Angus Robertson 577]

Your Win64 fixes are now in SVN thanks, with a couple more that you missed because they were LongInt.  Fortunately most PostMessages already casted pointers correctly, just these older samples. 

 

Angus