From 941ed2e6af0749032944cfe054bdf602c910e548 Mon Sep 17 00:00:00 2001 From: NeilBrown Date: Wed, 4 Oct 2023 17:00:33 +1100 Subject: [PATCH] rangetrack: fix some status returns. If the range name isn't registered yet, report 'false', not 'fail'. It might be a simple test to see if there is a registration. And rangetrack:clear should succeed if there is nothing to clear. Signed-off-by: NeilBrown --- lib-rangetrack.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib-rangetrack.c b/lib-rangetrack.c index bd47b00e..16d74654 100644 --- a/lib-rangetrack.c +++ b/lib-rangetrack.c @@ -110,7 +110,7 @@ DEF_CMD(rangetrack_add) struct mark *m, *m1, *m2; if (!i) - return Efail; + return Efalse; if (!start || !end) /* Testing if configured already */ return 1; @@ -182,7 +182,7 @@ DEF_CMD(rangetrack_clear) struct mark *m1, *m2; if (!i) - return Efail; + return Efalse; if (!start || !end) { start = vmark_first(ci->home, i->view, ci->home); end = vmark_last(ci->home, i->view, ci->home); @@ -234,7 +234,7 @@ DEF_CMD(rangetrack_clear) mark_step(m2, 0); } if (!m2) - return Efail; + return 1; /* m2 is now the end of an active section that needs to bie discarded */ while (m1 && mark_ordered_not_same(m1, m2)) { struct mark *m = m1; -- 2.39.5