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

Fix for issue open62541/open62541#6205: Different data types for namespace indexes used. #6289

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

sebschwinn
Copy link

Fix for issue #6205 Different data types for namespace indexes used.
Changed prototypes and functions for namespace methods from size_t to UA_UInt16. Also updated the affected tests.

@CLAassistant
Copy link

CLAassistant commented Feb 19, 2024

CLA assistant check
All committers have signed the CLA.

@sebschwinn
Copy link
Author

Hello all,
Unfortunately I am not able to evaluate the error of the CI-processes of msys64 (and clang) on my own. Can anybody support me? Or has anybody else seen a fix for this? The command includes the -NonInteractive flag, but the process still wants to read credentials, which is (as of my knowledge) not possible.
Greetings,
Sebastian

@jpfr
Copy link
Member

jpfr commented Feb 20, 2024

Thanks we'll look into it.

@mcpat
Copy link
Contributor

mcpat commented May 16, 2024

Is there a reason why you do not change the prototypes and implementations of the internal functions getNamespaceByIndex and getNamespaceByName? It would be great if the public and internal prototypes remain identical.

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

Successfully merging this pull request may close these issues.

None yet

5 participants