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

Consider postponing processing of PLC files #2

Open
RobertoRoos opened this issue Feb 9, 2023 · 1 comment
Open

Consider postponing processing of PLC files #2

RobertoRoos opened this issue Feb 9, 2023 · 1 comment

Comments

@RobertoRoos
Copy link
Member

Now all sources configured in conf.py are loaded from the start and put into a list to be searched in later.
This could be replaced by only indexing the files at the start and only loading them when they are called from an auto... method.

This might make plcdoc more robust and quick as no unnecessary processing is done.

@RobertoRoos
Copy link
Member Author

RobertoRoos commented Feb 9, 2023

One problem: an object has to be correlated with a file name, but only the declaration inside the files are leading. So the XML should be already be processed too.

EDIT: Actually, Visual Studio will automatically rename files to match with their internal declaration, so only relying on the file name would be fine.
To my knowledge there are never multiple declarations for a single file.

EDIT2: Visual Studio gives an error trying to open a file with more than one declaration. And TwinCAT fails to compile it.

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

1 participant