Skip to content

fix(partition): simplify redundant condition in chart drawing#205

Merged
deepin-bot[bot] merged 1 commit into
linuxdeepin:masterfrom
GongHeng2017:202606011446-master-fix
Jun 1, 2026
Merged

fix(partition): simplify redundant condition in chart drawing#205
deepin-bot[bot] merged 1 commit into
linuxdeepin:masterfrom
GongHeng2017:202606011446-master-fix

Conversation

@GongHeng2017
Copy link
Copy Markdown
Contributor

Remove redundant else if (m_sumValue >= 100) condition check, simplify to else since the preceding if already covers the complementary case.

简化分区图表绘制中冗余的条件判断,将 else if 简化为 else。

Log: 简化分区图表绘制冗余条件
PMS: https://pms.uniontech.com/bug-view-354861.html
Influence: 仅代码逻辑简化,不影响功能行为。

Remove redundant `else if (m_sumValue >= 100)` condition check,
simplify to `else` since the preceding `if` already covers the
complementary case.

简化分区图表绘制中冗余的条件判断,将 else if 简化为 else。

Log: 简化分区图表绘制冗余条件
PMS: https://pms.uniontech.com/bug-view-354861.html
Influence: 仅代码逻辑简化,不影响功能行为。
Copy link
Copy Markdown

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry @GongHeng2017, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@GongHeng2017
Copy link
Copy Markdown
Contributor Author

/forcemerge

@deepin-bot
Copy link
Copy Markdown
Contributor

deepin-bot Bot commented Jun 1, 2026

This pr force merged! (status: unstable)

@deepin-bot deepin-bot Bot merged commit 6f93b82 into linuxdeepin:master Jun 1, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants