psu: fix check for invalid timeout value - Githubissues

phoenix-rtos / phoenix-rtos-hostutils

Phoenix-RTOS supporting daemon

BSD 3-Clause "New" or "Revised" License

1 stars 5 forks source link

psu: fix check for invalid timeout value #46

Closed gerard5 closed 1 year ago

gerard5 commented 1 year ago

Quick fix for https://github.com/phoenix-rtos/phoenix-rtos-hostutils/pull/45#discussion_r1241262716

Fixes invalid value detection case, e.g.: psu -t 10abc script.plo psu -t "10 xyz 1234" script.plo

Distinguishes as an error: psu -t "" script.plo

JIRA: RTOS-510

Description

Motivation and Context

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

How Has This Been Tested?

[ ] Already covered by automatic testing.
[ ] New test added: (add PR link here).
[ ] Tested by hand on: (list targets here).

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing linter checks and tests passed.
[ ] My changes generate no new compilation warnings for any of the targets.

Special treatment

[ ] This PR needs additional PRs to work (list the PRs, preferably in merge-order).
[ ] I will merge this PR by myself when appropriate.