已合并
【PR】: 简要描述 修改全局变量的作用域 #1947
【PR】: 简要描述 修改全局变量的作用域 #1947
已合并
jiang-mingming01创建于 5月6日
5月6日

Pull Request

描述

请清晰准确地描述本次 Pull Request 的意图和变更内容。
1、全局单例的变量整改成函数的局部静态变量
2、使用MAKE_SHARED宏保护申请动作
3、新增ini配置文件

变更类型

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

关联的Issue

如何测试

描述测试此变更的步骤和前提条件:
调用PlatFormInfos::InitByInstance()初始化全局单例
正常执行业务逻辑
进程正常退出
使用ASAN检测内存泄漏
观察泄漏报告:PlatFormInfosImpl对象未释放

简易代码
PlatFormInfos info;
info.InitByInstance();
// 进程退出

核对清单

其他信息

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

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 jiang-mingming01 的贡献)
Jjiang-mingming01
5月6日 创建了 pull request,commit e8a55da5
Jjiang-mingming01
5月6日 添加了label:enhancement
Jjiang-mingming01
5月6日 关联了issue:[Bug-Report|缺陷反馈]:
CANN-robot
CANN-robot成员
5月6日 评论:

Thanks for your pull-request.
The full list of commands accepted by me can be found at here
You can get sig-info at here


PR Approval Progress

Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
repo-cann/runtime 张子菁, 王涛 (2/2) 张子菁 (1/1)
src/platform jason_yuan_ye, arthur_morgan_hw, 王涛 (3/2) jason_yuan_ye (1/1)
tests/ut/platform arthur_morgan_hw, 王涛, jason_yuan_ye (3/2) jason_yuan_ye (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

jiang-mingming01, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
CANN-robotCANN-robot成员
5月6日 添加了label:cann-cla/yes
此处折叠了93条消息 查看更多
李俊谕成员
5月12日 解决了最后一个问题
tingwood成员
5月12日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
5月12日 添加了label:lgtmapproved
CANN-robot
CANN-robot成员
5月12日 评论:

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: jason_yuan_ye, 张子菁.
Reviewers who wrote a comment of /lgtm are: jason_yuan_ye, arthur_morgan_hw, 张子菁, 王涛.

likedislike
CANN-robotCANN-robot成员
5月12日 合入了pull request