Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CHERRY-PICK] Fix while loop in delegated event API #309

Merged
merged 1 commit into from
Nov 15, 2024

Conversation

kuqin12
Copy link
Contributor

@kuqin12 kuqin12 commented Nov 15, 2024

Description

This commit introduced the bug
de3cf0c#diff-516242aa392b05a93f0222ff1e534b10a1f905124a16abd5704fa938d6590b48R143 where the swtch case and if loop for return status were moved outside the while loop

For details on how to complete these options and their meaning refer to CONTRIBUTING.md.

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?
  • Backport to release branch?

How This Was Tested

Tested on arm platform

Integration Instructions

N/A

(cherry picked from commit bd20dd8)

## Description

this commit introduced the bug
microsoft@de3cf0c#diff-516242aa392b05a93f0222ff1e534b10a1f905124a16abd5704fa938d6590b48R143
where the swtch case and if loop for return status were moved outside
the while loop

For details on how to complete these options and their meaning refer to
[CONTRIBUTING.md](https://github.com/microsoft/mu/blob/HEAD/CONTRIBUTING.md).

- [x] Impacts functionality?
- [ ] Impacts security?
- [ ] Breaking change?
- [ ] Includes tests?
- [ ] Includes documentation?
- [ ] Backport to release branch?

## How This Was Tested

Tested on arm platform

## Integration Instructions

N/A

(cherry picked from commit bd20dd8)
@kuqin12 kuqin12 requested review from os-d, cfernald and apop5 November 15, 2024 23:37
@apop5 apop5 merged commit f1daef5 into microsoft:release/202405 Nov 15, 2024
9 checks passed
@kuqin12 kuqin12 deleted the cherry-pick branch November 15, 2024 23:56
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.

3 participants