]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
test-list: rename field to indicate that it's a list 23855/head
authorZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Wed, 29 Jun 2022 12:46:47 +0000 (14:46 +0200)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Sat, 2 Jul 2022 10:51:37 +0000 (12:51 +0200)
With this commit, lists are generally either names with plural (items),
or in a way that indicates a multi-item container (queue, *_list, or similar).

src/test/test-list.c

index dc7ecb7572e94f9071adfe5ab9905c2d6267591b..2c764d7b71127fa7588a71ab51768290d23a48a7 100644 (file)
@@ -6,7 +6,7 @@
 int main(int argc, const char *argv[]) {
         size_t i;
         typedef struct list_item {
-                LIST_FIELDS(struct list_item, item);
+                LIST_FIELDS(struct list_item, item_list);
         } list_item;
         LIST_HEAD(list_item, head);
         LIST_HEAD(list_item, head2);
@@ -18,243 +18,243 @@ int main(int argc, const char *argv[]) {
         assert_se(head2 == NULL);
 
         for (i = 0; i < ELEMENTSOF(items); i++) {
-                LIST_INIT(item, &items[i]);
-                assert_se(LIST_JUST_US(item, &items[i]));
-                LIST_PREPEND(item, head, &items[i]);
+                LIST_INIT(item_list, &items[i]);
+                assert_se(LIST_JUST_US(item_list, &items[i]));
+                LIST_PREPEND(item_list, head, &items[i]);
         }
 
         i = 0;
-        LIST_FOREACH_OTHERS(item, cursor, &items[2]) {
+        LIST_FOREACH_OTHERS(item_list, cursor, &items[2]) {
                 i++;
                 assert_se(cursor != &items[2]);
         }
         assert_se(i == ELEMENTSOF(items)-1);
 
         i = 0;
-        LIST_FOREACH_OTHERS(item, cursor, &items[0]) {
+        LIST_FOREACH_OTHERS(item_list, cursor, &items[0]) {
                 i++;
                 assert_se(cursor != &items[0]);
         }
         assert_se(i == ELEMENTSOF(items)-1);
 
         i = 0;
-        LIST_FOREACH_OTHERS(item, cursor, &items[3]) {
+        LIST_FOREACH_OTHERS(item_list, cursor, &items[3]) {
                 i++;
                 assert_se(cursor != &items[3]);
         }
         assert_se(i == ELEMENTSOF(items)-1);
 
-        assert_se(!LIST_JUST_US(item, head));
+        assert_se(!LIST_JUST_US(item_list, head));
 
-        assert_se(items[0].item_next == NULL);
-        assert_se(items[1].item_next == &items[0]);
-        assert_se(items[2].item_next == &items[1]);
-        assert_se(items[3].item_next == &items[2]);
+        assert_se(items[0].item_list_next == NULL);
+        assert_se(items[1].item_list_next == &items[0]);
+        assert_se(items[2].item_list_next == &items[1]);
+        assert_se(items[3].item_list_next == &items[2]);
 
-        assert_se(items[0].item_prev == &items[1]);
-        assert_se(items[1].item_prev == &items[2]);
-        assert_se(items[2].item_prev == &items[3]);
-        assert_se(items[3].item_prev == NULL);
+        assert_se(items[0].item_list_prev == &items[1]);
+        assert_se(items[1].item_list_prev == &items[2]);
+        assert_se(items[2].item_list_prev == &items[3]);
+        assert_se(items[3].item_list_prev == NULL);
 
         list_item *cursor;
-        LIST_FIND_HEAD(item, &items[0], cursor);
+        LIST_FIND_HEAD(item_list, &items[0], cursor);
         assert_se(cursor == &items[3]);
 
-        LIST_FIND_TAIL(item, &items[3], cursor);
+        LIST_FIND_TAIL(item_list, &items[3], cursor);
         assert_se(cursor == &items[0]);
 
-        LIST_REMOVE(item, head, &items[1]);
-        assert_se(LIST_JUST_US(item, &items[1]));
-
-        assert_se(items[0].item_next == NULL);
-        assert_se(items[2].item_next == &items[0]);
-        assert_se(items[3].item_next == &items[2]);
-
-        assert_se(items[0].item_prev == &items[2]);
-        assert_se(items[2].item_prev == &items[3]);
-        assert_se(items[3].item_prev == NULL);
-
-        LIST_INSERT_AFTER(item, head, &items[3], &items[1]);
-        assert_se(items[0].item_next == NULL);
-        assert_se(items[2].item_next == &items[0]);
-        assert_se(items[1].item_next == &items[2]);
-        assert_se(items[3].item_next == &items[1]);
-
-        assert_se(items[0].item_prev == &items[2]);
-        assert_se(items[2].item_prev == &items[1]);
-        assert_se(items[1].item_prev == &items[3]);
-        assert_se(items[3].item_prev == NULL);
-
-        LIST_REMOVE(item, head, &items[1]);
-        assert_se(LIST_JUST_US(item, &items[1]));
-
-        assert_se(items[0].item_next == NULL);
-        assert_se(items[2].item_next == &items[0]);
-        assert_se(items[3].item_next == &items[2]);
-
-        assert_se(items[0].item_prev == &items[2]);
-        assert_se(items[2].item_prev == &items[3]);
-        assert_se(items[3].item_prev == NULL);
-
-        LIST_INSERT_BEFORE(item, head, &items[2], &items[1]);
-        assert_se(items[0].item_next == NULL);
-        assert_se(items[2].item_next == &items[0]);
-        assert_se(items[1].item_next == &items[2]);
-        assert_se(items[3].item_next == &items[1]);
-
-        assert_se(items[0].item_prev == &items[2]);
-        assert_se(items[2].item_prev == &items[1]);
-        assert_se(items[1].item_prev == &items[3]);
-        assert_se(items[3].item_prev == NULL);
-
-        LIST_REMOVE(item, head, &items[0]);
-        assert_se(LIST_JUST_US(item, &items[0]));
-
-        assert_se(items[2].item_next == NULL);
-        assert_se(items[1].item_next == &items[2]);
-        assert_se(items[3].item_next == &items[1]);
-
-        assert_se(items[2].item_prev == &items[1]);
-        assert_se(items[1].item_prev == &items[3]);
-        assert_se(items[3].item_prev == NULL);
-
-        LIST_INSERT_BEFORE(item, head, &items[3], &items[0]);
-        assert_se(items[2].item_next == NULL);
-        assert_se(items[1].item_next == &items[2]);
-        assert_se(items[3].item_next == &items[1]);
-        assert_se(items[0].item_next == &items[3]);
-
-        assert_se(items[2].item_prev == &items[1]);
-        assert_se(items[1].item_prev == &items[3]);
-        assert_se(items[3].item_prev == &items[0]);
-        assert_se(items[0].item_prev == NULL);
+        LIST_REMOVE(item_list, head, &items[1]);
+        assert_se(LIST_JUST_US(item_list, &items[1]));
+
+        assert_se(items[0].item_list_next == NULL);
+        assert_se(items[2].item_list_next == &items[0]);
+        assert_se(items[3].item_list_next == &items[2]);
+
+        assert_se(items[0].item_list_prev == &items[2]);
+        assert_se(items[2].item_list_prev == &items[3]);
+        assert_se(items[3].item_list_prev == NULL);
+
+        LIST_INSERT_AFTER(item_list, head, &items[3], &items[1]);
+        assert_se(items[0].item_list_next == NULL);
+        assert_se(items[2].item_list_next == &items[0]);
+        assert_se(items[1].item_list_next == &items[2]);
+        assert_se(items[3].item_list_next == &items[1]);
+
+        assert_se(items[0].item_list_prev == &items[2]);
+        assert_se(items[2].item_list_prev == &items[1]);
+        assert_se(items[1].item_list_prev == &items[3]);
+        assert_se(items[3].item_list_prev == NULL);
+
+        LIST_REMOVE(item_list, head, &items[1]);
+        assert_se(LIST_JUST_US(item_list, &items[1]));
+
+        assert_se(items[0].item_list_next == NULL);
+        assert_se(items[2].item_list_next == &items[0]);
+        assert_se(items[3].item_list_next == &items[2]);
+
+        assert_se(items[0].item_list_prev == &items[2]);
+        assert_se(items[2].item_list_prev == &items[3]);
+        assert_se(items[3].item_list_prev == NULL);
+
+        LIST_INSERT_BEFORE(item_list, head, &items[2], &items[1]);
+        assert_se(items[0].item_list_next == NULL);
+        assert_se(items[2].item_list_next == &items[0]);
+        assert_se(items[1].item_list_next == &items[2]);
+        assert_se(items[3].item_list_next == &items[1]);
+
+        assert_se(items[0].item_list_prev == &items[2]);
+        assert_se(items[2].item_list_prev == &items[1]);
+        assert_se(items[1].item_list_prev == &items[3]);
+        assert_se(items[3].item_list_prev == NULL);
+
+        LIST_REMOVE(item_list, head, &items[0]);
+        assert_se(LIST_JUST_US(item_list, &items[0]));
+
+        assert_se(items[2].item_list_next == NULL);
+        assert_se(items[1].item_list_next == &items[2]);
+        assert_se(items[3].item_list_next == &items[1]);
+
+        assert_se(items[2].item_list_prev == &items[1]);
+        assert_se(items[1].item_list_prev == &items[3]);
+        assert_se(items[3].item_list_prev == NULL);
+
+        LIST_INSERT_BEFORE(item_list, head, &items[3], &items[0]);
+        assert_se(items[2].item_list_next == NULL);
+        assert_se(items[1].item_list_next == &items[2]);
+        assert_se(items[3].item_list_next == &items[1]);
+        assert_se(items[0].item_list_next == &items[3]);
+
+        assert_se(items[2].item_list_prev == &items[1]);
+        assert_se(items[1].item_list_prev == &items[3]);
+        assert_se(items[3].item_list_prev == &items[0]);
+        assert_se(items[0].item_list_prev == NULL);
         assert_se(head == &items[0]);
 
-        LIST_REMOVE(item, head, &items[0]);
-        assert_se(LIST_JUST_US(item, &items[0]));
+        LIST_REMOVE(item_list, head, &items[0]);
+        assert_se(LIST_JUST_US(item_list, &items[0]));
 
-        assert_se(items[2].item_next == NULL);
-        assert_se(items[1].item_next == &items[2]);
-        assert_se(items[3].item_next == &items[1]);
+        assert_se(items[2].item_list_next == NULL);
+        assert_se(items[1].item_list_next == &items[2]);
+        assert_se(items[3].item_list_next == &items[1]);
 
-        assert_se(items[2].item_prev == &items[1]);
-        assert_se(items[1].item_prev == &items[3]);
-        assert_se(items[3].item_prev == NULL);
+        assert_se(items[2].item_list_prev == &items[1]);
+        assert_se(items[1].item_list_prev == &items[3]);
+        assert_se(items[3].item_list_prev == NULL);
 
-        LIST_INSERT_BEFORE(item, head, NULL, &items[0]);
-        assert_se(items[0].item_next == NULL);
-        assert_se(items[2].item_next == &items[0]);
-        assert_se(items[1].item_next == &items[2]);
-        assert_se(items[3].item_next == &items[1]);
+        LIST_INSERT_BEFORE(item_list, head, NULL, &items[0]);
+        assert_se(items[0].item_list_next == NULL);
+        assert_se(items[2].item_list_next == &items[0]);
+        assert_se(items[1].item_list_next == &items[2]);
+        assert_se(items[3].item_list_next == &items[1]);
 
-        assert_se(items[0].item_prev == &items[2]);
-        assert_se(items[2].item_prev == &items[1]);
-        assert_se(items[1].item_prev == &items[3]);
-        assert_se(items[3].item_prev == NULL);
+        assert_se(items[0].item_list_prev == &items[2]);
+        assert_se(items[2].item_list_prev == &items[1]);
+        assert_se(items[1].item_list_prev == &items[3]);
+        assert_se(items[3].item_list_prev == NULL);
 
-        LIST_REMOVE(item, head, &items[0]);
-        assert_se(LIST_JUST_US(item, &items[0]));
+        LIST_REMOVE(item_list, head, &items[0]);
+        assert_se(LIST_JUST_US(item_list, &items[0]));
 
-        assert_se(items[2].item_next == NULL);
-        assert_se(items[1].item_next == &items[2]);
-        assert_se(items[3].item_next == &items[1]);
+        assert_se(items[2].item_list_next == NULL);
+        assert_se(items[1].item_list_next == &items[2]);
+        assert_se(items[3].item_list_next == &items[1]);
 
-        assert_se(items[2].item_prev == &items[1]);
-        assert_se(items[1].item_prev == &items[3]);
-        assert_se(items[3].item_prev == NULL);
+        assert_se(items[2].item_list_prev == &items[1]);
+        assert_se(items[1].item_list_prev == &items[3]);
+        assert_se(items[3].item_list_prev == NULL);
 
-        LIST_REMOVE(item, head, &items[1]);
-        assert_se(LIST_JUST_US(item, &items[1]));
+        LIST_REMOVE(item_list, head, &items[1]);
+        assert_se(LIST_JUST_US(item_list, &items[1]));
 
-        assert_se(items[2].item_next == NULL);
-        assert_se(items[3].item_next == &items[2]);
+        assert_se(items[2].item_list_next == NULL);
+        assert_se(items[3].item_list_next == &items[2]);
 
-        assert_se(items[2].item_prev == &items[3]);
-        assert_se(items[3].item_prev == NULL);
+        assert_se(items[2].item_list_prev == &items[3]);
+        assert_se(items[3].item_list_prev == NULL);
 
-        LIST_REMOVE(item, head, &items[2]);
-        assert_se(LIST_JUST_US(item, &items[2]));
-        assert_se(LIST_JUST_US(item, head));
+        LIST_REMOVE(item_list, head, &items[2]);
+        assert_se(LIST_JUST_US(item_list, &items[2]));
+        assert_se(LIST_JUST_US(item_list, head));
 
-        LIST_REMOVE(item, head, &items[3]);
-        assert_se(LIST_JUST_US(item, &items[3]));
+        LIST_REMOVE(item_list, head, &items[3]);
+        assert_se(LIST_JUST_US(item_list, &items[3]));
 
         assert_se(head == NULL);
 
         for (i = 0; i < ELEMENTSOF(items); i++) {
-                assert_se(LIST_JUST_US(item, &items[i]));
-                LIST_APPEND(item, head, &items[i]);
+                assert_se(LIST_JUST_US(item_list, &items[i]));
+                LIST_APPEND(item_list, head, &items[i]);
         }
 
-        assert_se(!LIST_JUST_US(item, head));
+        assert_se(!LIST_JUST_US(item_list, head));
 
-        assert_se(items[0].item_next == &items[1]);
-        assert_se(items[1].item_next == &items[2]);
-        assert_se(items[2].item_next == &items[3]);
-        assert_se(items[3].item_next == NULL);
+        assert_se(items[0].item_list_next == &items[1]);
+        assert_se(items[1].item_list_next == &items[2]);
+        assert_se(items[2].item_list_next == &items[3]);
+        assert_se(items[3].item_list_next == NULL);
 
-        assert_se(items[0].item_prev == NULL);
-        assert_se(items[1].item_prev == &items[0]);
-        assert_se(items[2].item_prev == &items[1]);
-        assert_se(items[3].item_prev == &items[2]);
+        assert_se(items[0].item_list_prev == NULL);
+        assert_se(items[1].item_list_prev == &items[0]);
+        assert_se(items[2].item_list_prev == &items[1]);
+        assert_se(items[3].item_list_prev == &items[2]);
 
         for (i = 0; i < ELEMENTSOF(items); i++)
-                LIST_REMOVE(item, head, &items[i]);
+                LIST_REMOVE(item_list, head, &items[i]);
 
         assert_se(head == NULL);
 
         for (i = 0; i < ELEMENTSOF(items) / 2; i++) {
-                LIST_INIT(item, &items[i]);
-                assert_se(LIST_JUST_US(item, &items[i]));
-                LIST_PREPEND(item, head, &items[i]);
+                LIST_INIT(item_list, &items[i]);
+                assert_se(LIST_JUST_US(item_list, &items[i]));
+                LIST_PREPEND(item_list, head, &items[i]);
         }
 
         for (i = ELEMENTSOF(items) / 2; i < ELEMENTSOF(items); i++) {
-                LIST_INIT(item, &items[i]);
-                assert_se(LIST_JUST_US(item, &items[i]));
-                LIST_PREPEND(item, head2, &items[i]);
+                LIST_INIT(item_list, &items[i]);
+                assert_se(LIST_JUST_US(item_list, &items[i]));
+                LIST_PREPEND(item_list, head2, &items[i]);
         }
 
-        assert_se(items[0].item_next == NULL);
-        assert_se(items[1].item_next == &items[0]);
-        assert_se(items[2].item_next == NULL);
-        assert_se(items[3].item_next == &items[2]);
+        assert_se(items[0].item_list_next == NULL);
+        assert_se(items[1].item_list_next == &items[0]);
+        assert_se(items[2].item_list_next == NULL);
+        assert_se(items[3].item_list_next == &items[2]);
 
-        assert_se(items[0].item_prev == &items[1]);
-        assert_se(items[1].item_prev == NULL);
-        assert_se(items[2].item_prev == &items[3]);
-        assert_se(items[3].item_prev == NULL);
+        assert_se(items[0].item_list_prev == &items[1]);
+        assert_se(items[1].item_list_prev == NULL);
+        assert_se(items[2].item_list_prev == &items[3]);
+        assert_se(items[3].item_list_prev == NULL);
 
-        LIST_JOIN(item, head2, head);
+        LIST_JOIN(item_list, head2, head);
         assert_se(head == NULL);
 
-        assert_se(items[0].item_next == NULL);
-        assert_se(items[1].item_next == &items[0]);
-        assert_se(items[2].item_next == &items[1]);
-        assert_se(items[3].item_next == &items[2]);
+        assert_se(items[0].item_list_next == NULL);
+        assert_se(items[1].item_list_next == &items[0]);
+        assert_se(items[2].item_list_next == &items[1]);
+        assert_se(items[3].item_list_next == &items[2]);
 
-        assert_se(items[0].item_prev == &items[1]);
-        assert_se(items[1].item_prev == &items[2]);
-        assert_se(items[2].item_prev == &items[3]);
-        assert_se(items[3].item_prev == NULL);
+        assert_se(items[0].item_list_prev == &items[1]);
+        assert_se(items[1].item_list_prev == &items[2]);
+        assert_se(items[2].item_list_prev == &items[3]);
+        assert_se(items[3].item_list_prev == NULL);
 
-        LIST_JOIN(item, head, head2);
+        LIST_JOIN(item_list, head, head2);
         assert_se(head2 == NULL);
         assert_se(head);
 
         for (i = 0; i < ELEMENTSOF(items); i++)
-                LIST_REMOVE(item, head, &items[i]);
+                LIST_REMOVE(item_list, head, &items[i]);
 
         assert_se(head == NULL);
 
-        LIST_PREPEND(item, head, items + 0);
-        LIST_PREPEND(item, head, items + 1);
-        LIST_PREPEND(item, head, items + 2);
+        LIST_PREPEND(item_list, head, items + 0);
+        LIST_PREPEND(item_list, head, items + 1);
+        LIST_PREPEND(item_list, head, items + 2);
 
-        assert_se(LIST_POP(item, head) == items + 2);
-        assert_se(LIST_POP(item, head) == items + 1);
-        assert_se(LIST_POP(item, head) == items + 0);
-        assert_se(LIST_POP(item, head) == NULL);
+        assert_se(LIST_POP(item_list, head) == items + 2);
+        assert_se(LIST_POP(item_list, head) == items + 1);
+        assert_se(LIST_POP(item_list, head) == items + 0);
+        assert_se(LIST_POP(item_list, head) == NULL);
 
         return 0;
 }