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

更新 OPC UA 字符串字面量到 Variable 转换构造函数的写法 #228

Merged
merged 1 commit into from
Feb 16, 2025

Conversation

zhaoxi-scut
Copy link
Member

Pull Request 合并请求准备清单

详情参见此处

  • 我同意在 Apache 2 开源许可下为本项目做贡献
  • 此 pull request 是在正确的分支上提出的
  • 此 pull request 有对应的错误报告或其他待改进的内容
  • 我本地的 RMVL 进行了单元测试、性能测试,有对应的测试数据
  • 我提交的 feature 有很好的文档记录,并且可以使用 CMake 项目构建示例代码

具体内容

原先

rm::Variable v1 = std::string("xxx");

rm::Variable v2 = "xxx";

在经过 v1 == v2 的比较之后,返回的是 false,原因是字符串字面量到 rm::Variable 的构造函数的字符串长度是字符串字面量数组长度,而非字符串长度(区别在于是否考虑尾置 \0),现予以统一。

@zhaoxi-scut zhaoxi-scut added bug 代码上、功能上存在严重漏洞... optimization 代码重构、功能优化 labels Feb 16, 2025
@zhaoxi-scut zhaoxi-scut added this to the 2.3.0 milestone Feb 16, 2025
@zhaoxi-scut zhaoxi-scut merged commit 38bb940 into cv-rmvl:2.x Feb 16, 2025
8 checks passed
@zhaoxi-scut zhaoxi-scut deleted the opcua-event-dev branch February 16, 2025 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 代码上、功能上存在严重漏洞... optimization 代码重构、功能优化
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants