Skip to content

[HUST CSE] fix: Variable not initialized#1977

Open
Shelton-Z wants to merge 1 commit intoalibaba:masterfrom
Shelton-Z:fix-variable
Open

[HUST CSE] fix: Variable not initialized#1977
Shelton-Z wants to merge 1 commit intoalibaba:masterfrom
Shelton-Z:fix-variable

Conversation

@Shelton-Z
Copy link

@Shelton-Z Shelton-Z commented Apr 15, 2023

拉取/合并请求描述:(PR description)

为什么提交这份PR (why to submit this PR)

部分变量未赋初值(三处)

存在问题的文件:

  1. components/amp_adapter/platform/linux/peripheral/aos_hal_lcd.c
  2. hardware/chip/rtl872xd/sdk/component/common/mbed/targets/hal/rtl8721d/i2c_api.c

你的解决方案是什么 (what is your solution)

将未赋初值的变量直接赋值为0

image-20230415142154549

在什么测试环境下测试通过 (what is the test environment)

VSCode

当前拉取/合并请求的状态 Intent for your PR

必须选择一项 Choose one (Mandatory):

  • 本拉取/合并请求是一个草稿版本 This PR is for a code-review and is intended to get feedback
  • 本拉取/合并请求是一个成熟版本 This PR is mature, and ready to be integrated into the repo
aos_hal_lcd.c
i2c_api.c
@CLAassistant
Copy link

CLAassistant commented Apr 15, 2023

CLA assistant check
All committers have signed the CLA.

@Shelton-Z Shelton-Z marked this pull request as ready for review April 15, 2023 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants