-
Notifications
You must be signed in to change notification settings - Fork 37
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
Update OPC DA #65
Update OPC DA #65
Conversation
kumajaya
commented
Mar 31, 2024
•
edited
Loading
edited
- Don't do any unnecessary preventions for string conversion problems
- Remove items and groups depend on FB config
- Code refactoring from char* to std::string
- Proper memory usage and deallocation
Change SOpcItemData structure really fix unpredictable string conversion problems. Don't do any further unnecessary preventions. This commit also adds a copyright statement to the corresponding changed files.
It is possible to create an FB without a read group or without a write group.
@azoitl @MartinMelikMerkumians
Reference: https://grapeprogrammer.com/const_string_reference_cpp/ |
Yes this is my understanding of const references to std::string. There is an exception to this rule: When C++ compiler could apply the move semantics to transfer the string's content into a newly created object then a pass by value is more efficient. Am I right @MartinMelikMerkumians ? |