-
Notifications
You must be signed in to change notification settings - Fork 244
Description
Is your feature request related to a problem? Please describe.
Two minor redundant logic/checks:
osal/src/os/vxworks/src/os-impl-timebase.c
Lines 397 to 411 in ff4f523
| for (signo = SIGRTMIN; signo <= SIGRTMAX; ++signo) | |
| { | |
| if (!sigismember(&inuse, signo)) | |
| { | |
| /* signal is available, stop search */ | |
| break; | |
| } | |
| } | |
| if (signo < SIGRTMIN || signo > SIGRTMAX) | |
| { | |
| /* no available signal for timer */ | |
| OS_DEBUG("No free RT signals to use for simulated time base\n"); | |
| return_code = OS_TIMER_ERR_UNAVAILABLE; | |
| } |
osal/src/os/vxworks/src/os-impl-network.c
Line 88 in ff4f523
| status = OS_SUCCESS; |
No actual issues, just minor cleanup.
Describe the solution you'd like
Remove first, refactor second so the bound is just checked in one place
Describe alternatives you've considered
None
Additional context
Static analysis warnings
Requester Info
Jacob Hageman - NASA