已合并
fix: 清理clean code告警 #4673
likun104创建于 9月1日
fix: 清理clean code告警 #4673
已合并
likun104创建于 9月1日
likun104成员
9月1日

Pull Request

描述

清理clean code告警

变更类型

请选择本次引入的变更类型:

关联的Issue

如何测试

描述测试此变更的步骤和前提条件:

  1. 流水线跑通过,且rdv跑通过

核对清单

其他信息

在此添加任何其他关于本次 PR 的说明。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 likun104 的贡献)
Llikun104成员
9月1日 添加了label:enhancement
Llikun104成员
9月1日 创建了 pull request,commit e89e8b48
atomgit-bot
atomgit-bot
9月1日 评论:

变更摘要

本 PR 用于清理 clean code 告警,属于纯命名层面的调整,不涉及任何功能逻辑变更。核心改动是将 RegisterMultiFunc 函数中接收 MULTI_FUNC_WITH_Q_CREATOR_FUNC 类型参数的函数重载,其形参名由 funcWithQCreator 统一重命名为 funcCreator,使同一函数族中不同重载的形参命名保持一致,从而消除命名不一致引发的代码规范告警。

主要改动

  • 参数重命名(实现文件): 在 dflow/udf/flow_func/flow_func_manager.cpp 中,将 RegisterMultiFunc 重载(接收 MULTI_FUNC_WITH_Q_CREATOR_FUNC)的形参 funcWithQCreator 改为 funcCreator,并同步更新函数体内的空指针判断与 FlowFuncManager::Instance().Register(...) 调用,无行为变化。
  • 参数重命名(头文件声明): 在 dflow/udf/inc/external/flow_func/meta_multi_func.h 中,将同名重载的声明形参由 funcWithQCreator 改为 funcCreator,保持声明与实现一致。
likedislike
不准确?
atomgit-bot
atomgit-bot
9月1日 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
CANN-robotCANN-robot成员
9月1日 添加了label:cann-cla/yes
此处折叠了293条消息 查看更多
zhanj
zhanj成员
9月10日 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
9月10日 添加了label:lgtm
yangyongqiang
yangyongqiang成员
9月10日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
9月10日 添加了label:approved
CANN-robotCANN-robot成员
9月10日 合入了pull request