Code Upgrade To Work in Both (32-64 bit)

[hecgroups]

Hello Again,

I am using a script adopted from some website in year 2015 but I need to upgrade to work in both office 2007 and 2013.

Below is the code and copy of old version file is attached.

Please Login or Register  to view this content.

[Keebellah]

Hi, me here again this is all you need to change:

Please Login or Register  to view this content.

[MarvinP]

Hi hecgroups,

I've never heard of this before but some searching finds:
https://docs.microsoft.com/en-us/off...trsafe-keyword

Let me know if Hans suggestion worked.

[hecgroups]

Hey Keeb,

Thanks once again it works as I want let me check in both the versions and get back to you.
Thank once again for your help.

[Keebellah]

Let me know if Hans suggestion worked.

Hi Marvin,
I ran into similar issues when a friend I was helping ran into compatibility issues and he came up with this.
I've implemented it since and it was a lifesaver.

You can use this type if If Then Else conditions with the hash also to check if the OS is MAC or Windows and many more.

[hecgroups]

Thanks Keeb it works in both systems i.e. win 7 & 10 with office 2010 & 2013.

I will mark topic as solved and reputation add to you both.

[Keebellah]

Good to hear and thanks for the rep.
Happy coding

[romperstomper]

FYI, that code will not work properly in 64bit Office and is liable to crash.

[Keebellah]

I suggest you should be more specific about this, why and the caveats.

[romperstomper]

The caveat: if you get an API call wrong, it is liable to crash Excel without warning or chance to recover.

The why is that you just added PtrSafe which is not enough. For example, FindWindow returns a window handle, which is a Long in 32bit but a LongLong in 64bit, so it should be declared as LongPtr to allow it to return whichever is necessary. That also means that you can't change just the API lines - you also need to make sure that the variables are declared correctly depending on the version.

I do note though that the original post only mentions making the code work in both 2007 and 2013, whereas the title mentions 32 and 64bit, so it's not clear if the 2013 is, or may be, 64bit, or if it just needs to work in both versions at 32bit.

[romperstomper]

I don't have 64 bit here to test, but it should be something like this:

Please Login or Register  to view this content.