diff options
author | Tom Gundersen <teg@jklm.no> | 2013-10-26 13:41:31 +0200 |
---|---|---|
committer | Tom Gundersen <teg@jklm.no> | 2013-10-26 13:42:38 +0200 |
commit | 5ecb28f69ef670c7e194b656b57fda6f8850f0ba (patch) | |
tree | 4bc840bfa0150aad62947e0ad6c8ede5a719602a /src/shared | |
parent | 7d9cb4229cd5a47ab7dd50512fec018320294192 (diff) |
STRV_FOREACH_BACWARDS: start at the tail of the list
Otherwise, the user would have to manually initialize the pointer. Nobody currently uses this code,
so the change in behaviour sohuld be fine.
Diffstat (limited to 'src/shared')
-rw-r--r-- | src/shared/strv.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/shared/strv.h b/src/shared/strv.h index f6fb033a8c..571ea38a7f 100644 --- a/src/shared/strv.h +++ b/src/shared/strv.h @@ -75,7 +75,8 @@ bool strv_overlap(char **a, char **b) _pure_; for ((s) = (l); (s) && *(s); (s)++) #define STRV_FOREACH_BACKWARDS(s, l) \ - for (; (l) && ((s) >= (l)); (s)--) + STRV_FOREACH(s, l) ; \ + for ((s)--; (l) && ((s) >= (l)); (s)--) #define STRV_FOREACH_PAIR(x, y, l) \ for ((x) = (l), (y) = (x+1); (x) && *(x) && *(y); (x) += 2, (y) = (x + 1)) |