Fix bug: invalid return

Bug #1835872 reported by lihang on 2019-07-09
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
StarlingX
Low
lihang

Bug Description

Brief Description
-----------------
For the switch case statement, when there is a default branch, it will always execute the default branch if it does not match the case branch. Therefore, the return outside the switch function cannot be performed.
In order to improve the maintainability of the code, I agree this line of code should be removed. For this change, I did a verification on the system to ensure that it does not affect the functionality of the system.

lihang (lihang0166) on 2019-07-09
Changed in starlingx:
assignee: nobody → lihang (lihang0166)

Fix proposed to branch: master
Review: https://review.opendev.org/669797

Changed in starlingx:
status: New → In Progress
Ghada Khalil (gkhalil) wrote :

Low priority / not gating any stx release - minor code cleanup with no functional impact

tags: added: stx.ha
Changed in starlingx:
importance: Undecided → Low

Change abandoned by lihang (<email address hidden>) on branch: master
Review: https://review.opendev.org/669797

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers