Simplify some control flow code#24907
Conversation
|
@Deycoesr Please sign the Contributor License Agreement! Click here to manually synchronize the status of this Pull Request. See the FAQ for frequently asked questions. |
|
@Deycoesr Thank you for signing the Contributor License Agreement! |
sbrannen
left a comment
There was a problem hiding this comment.
Thanks for the PR!
I requested that you revert a few of the changes for the sake of readability.
Once you make those changes, I'll be glad to merge this PR.
There was a problem hiding this comment.
Although the proposed changed is technically correct, please revert this for the sake of readability.
There was a problem hiding this comment.
Although the proposed changed is technically correct, please revert this for the sake of readability.
There was a problem hiding this comment.
Although the proposed changed is technically correct, please revert this for the sake of readability.
|
@Deycoesr, did you intentionally close this PR? |
|
Not, I used |
|
I'm sorry, i don't know there will be such consequences. I created a new PR. #24925 |
I found some control flow code can be simplify