Jump to content
Fr0sT.Brutal

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

Recommended Posts

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

 

Share this post


Link to post
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,

 

Share this post


Link to post
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.

 

Share this post


Link to post

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

 

  • Thanks 1

Share this post


Link to post

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now
×