]> Kevux Git Server - fll/commit
Bugfix: Require is not properly triggering failure on error.
authorKevin Day <thekevinday@gmail.com>
Sun, 24 Oct 2021 03:42:34 +0000 (22:42 -0500)
committerKevin Day <thekevinday@gmail.com>
Sun, 24 Oct 2021 03:42:34 +0000 (22:42 -0500)
commitf2eb62222636b24cd9ad50025b5e18011ff06cc3
treea52579670cab2fb6dbe1847179bd46fb566fa3d5
parent2ced7549e9d678576e653c9764712e37e976ade4
Bugfix: Require is not properly triggering failure on error.

Get and handle the status code of controller_rule_wait_all().

Fix cases where "status" is being used where "status_lock" is better suited.

Remove unnecessary double assignment of status error.
In this case, only when F_lock is returned should the status be set.
This may need to be revisited and have the called function also set error on F_lock rather than handling here.

For asynchronous functions controller_rule_wait_all() is not properly handling the error.
This is happening because "required" is being incorrectly used.
For this function "required" is design to only wait for all required processes only and not limit the required validation.
Removing this fixing the problem so that when an asynchronous process that is required fails, the failure is propagated to the waiting process.

Add "require" in the htop example as a place for testing.

Add additional comments for clarifications and cleanup existing comments.

Remove unnecessary ready wait assignment.
level_3/controller/c/private-controller.c
level_3/controller/c/private-rule.c
level_3/controller/c/private-rule.h
level_3/controller/c/private-thread.c
level_3/controller/data/settings/example/entries/htop.entry