Skip to content

Delete duplicate code#314

Open
iysheng wants to merge 1 commit intoHyperInspire:masterfrom
iysheng:master
Open

Delete duplicate code#314
iysheng wants to merge 1 commit intoHyperInspire:masterfrom
iysheng:master

Conversation

@iysheng
Copy link

@iysheng iysheng commented Sep 25, 2025

I found these codes is duplicate as below:
image

So, I delete these code.

Copy link

@devloai devloai bot left a comment

Choose a reason for hiding this comment

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

PR Summary:

• Removes duplicate error handling code in AnyNetAdapter::LoadData method
• Eliminates redundant if-block that checked ret != InferenceWrapper::WrapperOk
• Preserves the necessary error checking logic that follows
• Clean refactoring with no functional changes

Review Summary:

This PR correctly identifies and removes duplicate error handling code without introducing any issues. The duplicate block was immediately followed by identical error checking logic, making the removal safe and beneficial for code maintainability. No critical issues found.

Follow-up suggestions:

@devloai scan for similar duplicate code patterns across the codebase

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.

1 participant