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

Cannot compile: Undeclared identifier 'PImageOptionalHeader' in unitPEFile #2

Open
ReinierNL opened this issue Jan 23, 2016 · 3 comments

Comments

@ReinierNL
Copy link

I'm not even sure if the code in this repository is meant to be compilable (I understand that it is mostly meant to not let this valuable code disappear completely).

But if it is:
I am trying to compile it now with Delphi 10 Seattle.
I've downloaded the prerequisites, made sure the compiler can find the source code, then also downloaded Virtual TreeView, and then I get stuck on this error message from the compiler:

Undeclared identifier 'PImageOptionalHeader' in unitPEFile

Is there code missing? Is this due to differences between Delphi versions?

@ReinierNL
Copy link
Author

It's due to differences between Delphi versions!
In WinApi.Windows we now have PImageOptionalHeader32 and PImageOptionalHeader64.
I'm taking PImageOptionalHeader32 for the time being.

Now there are other errors: PChar/PAnsiChar incompatibilities; cannot assign to left side; TntStdCtrls not found..

@ReinierNL
Copy link
Author

I got everything to compile, except MainForm, where I run into incompatibilities with VirtualTrees.
First of all, there is no unit ExVirtualStringTree. So I renamed TExVirtualStringTree to TExVirtualStringTree.
But now there are too many properties and methods missing, like Undo and Redo, NodeObject, ForeEach and more.
Which VirtualTrees version might work here?

@stefansundin
Copy link
Owner

If you figure out all the correct versions, please let me know and I'll put that information in the README file. If there are source code changes needed to increase compatibility, I think it's Ok to patch it, so feel free to create a pull request. Thanks!

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