-
Notifications
You must be signed in to change notification settings - Fork 29
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
fix: timer limited repeat #659
base: main
Are you sure you want to change the base?
Conversation
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.OpenSSF Scorecard
Scanned Manifest Files |
🦙 MegaLinter status:
|
Descriptor | Linter | Files | Fixed | Errors | Elapsed time |
---|---|---|---|---|---|
✅ ACTION | actionlint | 12 | 0 | 0.06s | |
✅ CPP | clang-format | 965 | 2 | 0 | 6.25s |
✅ DOCKERFILE | hadolint | 2 | 0 | 0.1s | |
✅ JSON | jsonlint | 9 | 0 | 0.15s | |
✅ JSON | prettier | 9 | 0 | 0 | 0.64s |
markdownlint | 6 | 0 | 4 | 1.52s | |
markdown-link-check | 6 | 1 | 78.5s | ||
✅ MARKDOWN | markdown-table-formatter | 6 | 0 | 0 | 0.32s |
✅ REPOSITORY | checkov | yes | no | 21.23s | |
✅ REPOSITORY | git_diff | yes | no | 0.07s | |
✅ REPOSITORY | grype | yes | no | 18.36s | |
✅ REPOSITORY | secretlint | yes | no | 7.34s | |
✅ REPOSITORY | trivy | yes | no | 12.2s | |
✅ REPOSITORY | trivy-sbom | yes | no | 0.65s | |
✅ REPOSITORY | trufflehog | yes | no | 2.95s | |
lychee | 139 | 4 | 3.27s | ||
prettier | 22 | 0 | 1 | 0.99s | |
✅ YAML | v8r | 22 | 0 | 9.54s | |
✅ YAML | yamllint | 22 | 0 | 0.41s |
See detailed report in MegaLinter reports
|
@@ -85,24 +85,34 @@ namespace infra | |||
void ComputeNextTriggerTime() override; | |||
}; | |||
|
|||
namespace details |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For consistency with other detail namespaces:
namespace details | |
namespace detail |
{ | ||
public: | ||
explicit TimerLimitedRepeatingWithClosingAction(uint32_t timerServiceId = systemTimerServiceId); | ||
TimerLimitedRepeatingWithClosingAction(int aHowMany, Duration duration, const infra::Function<void()>& action, const infra::Function<void()>& aClosingAction, uint32_t timerServiceId = systemTimerServiceId); | ||
|
||
void Start(int aHowMany, Duration duration, const infra::Function<void()>& action, const infra::Function<void()>& aClosingAction); | ||
virtual const infra::Function<void()>& Action() const override; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
virtual const infra::Function<void()>& Action() const override; | |
const infra::Function<void()>& Action() const override; |
TimerLimitedRepeating did not work on our project.
Refactored TimerLimitedRepeating and TimerLimitedRepeatingWithClosingAction to be aglined with TimerRepeating.
Added test cases for the 'Cancel'