Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

XLOPER12 #24

Open
igitur opened this issue Jun 11, 2021 · 3 comments
Open

XLOPER12 #24

igitur opened this issue Jun 11, 2021 · 3 comments

Comments

@igitur
Copy link

igitur commented Jun 11, 2021

Hi Eric,

I noticed that a formula like =ohPack(A1:F70000) returns #N/A. In fact, the largest row number to give a correct return value is =ohPack(A1:F65535).

I'm guessing this is an Excel4 vs Excel12 issue. I tried to make sense of the xlsdk and it seems in many places it's using XLOPER. Other places OPER is used, which typedefs back to XLOPER. I'm not sure whether the distinction there is relevant.

Do you think changing from XLOPER to XLOPER12 would fix the issue above? Of course this will break compatability with Excel 2003 and lower, but I think that would be OK.

@igitur
Copy link
Author

igitur commented Jul 14, 2021

I'd like to attempt to implement XLOPER12. It's not going to be easy for me. But let's try.

It looks like the original author implemented the original Excel SDK and then made some changes to it, for example the commenting out of TempStr and replacement by 2 other methods (TempStrNoSize and TempStrStl). I can't find the original Excel 15 (or earlier) SDK files anywhere, so it's difficult for me to see exactly what changes were made. If I implemented the new SDK, I have to make the same changes.

Do you perhaps have the original, unmodified SDK files or know exactly what changes were made to the interface?

Sorry, I'm not even sure whether I'm asking the correct questions as this is my first foray into Excel SDK development. But I'll do my best.

@eehlers
Copy link
Owner

eehlers commented Jul 16, 2021

There is a book by Steve Dalton which explains the Excel C API, you might find that helpful.

The SDK files were added to the repo. The TempStr* methods were implemented to fix a bug that was present in that version. I believe that the problem is fixed in newer versions of the SDK.

@igitur
Copy link
Author

igitur commented Jul 19, 2021

There is a book by Steve Dalton which explains the Excel C API, you might find that helpful.

Great, got it, thanks.

The SDK files were added to the repo. The TempStr* methods were implemented to fix a bug that was present in that version. I believe that the problem is fixed in newer versions of the SDK.

I meant the original unmodified SDK files, as distributed by Microsoft. But found them at http://download.microsoft.com/download/excel97win/Install/1.0/W9XNT4XP/EN-US/excel97sdk.exe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants