src/string_list.c

Sat, 04 Jul 2026 12:09:37 +0200

author
Mike Becker <universe@uap-core.de>
date
Sat, 04 Jul 2026 12:09:37 +0200
changeset 102
665b60727a89
parent 101
0cb645809b1a
child 103
31fa205db85a
permissions
-rw-r--r--

improve code and increase const-correctness

relates to #917

10
ecf787666f44 refactored sources
Mike Becker <universe@uap-core.de>
parents:
diff changeset
1 /*
34
fa9bda32de17 moved src files to src subdirectory and added licence text
Mike Becker <universe@uap-core.de>
parents: 20
diff changeset
2 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS HEADER.
57
68018eac46c3 adds simple tiny test suite and updates license headers
Mike Becker <universe@uap-core.de>
parents: 48
diff changeset
3 * Copyright 2018 Mike Becker. All rights reserved.
34
fa9bda32de17 moved src files to src subdirectory and added licence text
Mike Becker <universe@uap-core.de>
parents: 20
diff changeset
4 *
fa9bda32de17 moved src files to src subdirectory and added licence text
Mike Becker <universe@uap-core.de>
parents: 20
diff changeset
5 * Redistribution and use in source and binary forms, with or without
fa9bda32de17 moved src files to src subdirectory and added licence text
Mike Becker <universe@uap-core.de>
parents: 20
diff changeset
6 * modification, are permitted provided that the following conditions are met:
fa9bda32de17 moved src files to src subdirectory and added licence text
Mike Becker <universe@uap-core.de>
parents: 20
diff changeset
7 *
fa9bda32de17 moved src files to src subdirectory and added licence text
Mike Becker <universe@uap-core.de>
parents: 20
diff changeset
8 * 1. Redistributions of source code must retain the above copyright
fa9bda32de17 moved src files to src subdirectory and added licence text
Mike Becker <universe@uap-core.de>
parents: 20
diff changeset
9 * notice, this list of conditions and the following disclaimer.
fa9bda32de17 moved src files to src subdirectory and added licence text
Mike Becker <universe@uap-core.de>
parents: 20
diff changeset
10 *
fa9bda32de17 moved src files to src subdirectory and added licence text
Mike Becker <universe@uap-core.de>
parents: 20
diff changeset
11 * 2. Redistributions in binary form must reproduce the above copyright
fa9bda32de17 moved src files to src subdirectory and added licence text
Mike Becker <universe@uap-core.de>
parents: 20
diff changeset
12 * notice, this list of conditions and the following disclaimer in the
fa9bda32de17 moved src files to src subdirectory and added licence text
Mike Becker <universe@uap-core.de>
parents: 20
diff changeset
13 * documentation and/or other materials provided with the distribution.
fa9bda32de17 moved src files to src subdirectory and added licence text
Mike Becker <universe@uap-core.de>
parents: 20
diff changeset
14 *
fa9bda32de17 moved src files to src subdirectory and added licence text
Mike Becker <universe@uap-core.de>
parents: 20
diff changeset
15 * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
fa9bda32de17 moved src files to src subdirectory and added licence text
Mike Becker <universe@uap-core.de>
parents: 20
diff changeset
16 * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
fa9bda32de17 moved src files to src subdirectory and added licence text
Mike Becker <universe@uap-core.de>
parents: 20
diff changeset
17 * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
fa9bda32de17 moved src files to src subdirectory and added licence text
Mike Becker <universe@uap-core.de>
parents: 20
diff changeset
18 * DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE
fa9bda32de17 moved src files to src subdirectory and added licence text
Mike Becker <universe@uap-core.de>
parents: 20
diff changeset
19 * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
fa9bda32de17 moved src files to src subdirectory and added licence text
Mike Becker <universe@uap-core.de>
parents: 20
diff changeset
20 * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
fa9bda32de17 moved src files to src subdirectory and added licence text
Mike Becker <universe@uap-core.de>
parents: 20
diff changeset
21 * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
fa9bda32de17 moved src files to src subdirectory and added licence text
Mike Becker <universe@uap-core.de>
parents: 20
diff changeset
22 * CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
fa9bda32de17 moved src files to src subdirectory and added licence text
Mike Becker <universe@uap-core.de>
parents: 20
diff changeset
23 * OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
57
68018eac46c3 adds simple tiny test suite and updates license headers
Mike Becker <universe@uap-core.de>
parents: 48
diff changeset
24 * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
10
ecf787666f44 refactored sources
Mike Becker <universe@uap-core.de>
parents:
diff changeset
25 */
ecf787666f44 refactored sources
Mike Becker <universe@uap-core.de>
parents:
diff changeset
26
19
8bac9fd0629d generalized suffix_list to string_list
Mike Becker <universe@uap-core.de>
parents: 17
diff changeset
27 #include "string_list.h"
10
ecf787666f44 refactored sources
Mike Becker <universe@uap-core.de>
parents:
diff changeset
28
81
54515e5d60f5 fix ineffecient string list reallocation - fixes #637
Mike Becker <universe@uap-core.de>
parents: 57
diff changeset
29 #include <assert.h>
54515e5d60f5 fix ineffecient string list reallocation - fixes #637
Mike Becker <universe@uap-core.de>
parents: 57
diff changeset
30
102
665b60727a89 improve code and increase const-correctness
Mike Becker <universe@uap-core.de>
parents: 101
diff changeset
31 static void do_not_free(void *item) {
101
0cb645809b1a code improvements that avoid warnings on higher warning levels
Mike Becker <universe@uap-core.de>
parents: 99
diff changeset
32 (void) item; /* do not do anything with the item */
94
9637e3efb8e7 add element destructor to string_list
Mike Becker <universe@uap-core.de>
parents: 82
diff changeset
33 }
9637e3efb8e7 add element destructor to string_list
Mike Becker <universe@uap-core.de>
parents: 82
diff changeset
34
102
665b60727a89 improve code and increase const-correctness
Mike Becker <universe@uap-core.de>
parents: 101
diff changeset
35 string_list *new_string_list() {
665b60727a89 improve code and increase const-correctness
Mike Becker <universe@uap-core.de>
parents: 101
diff changeset
36 string_list *stringList = malloc(sizeof(string_list));
19
8bac9fd0629d generalized suffix_list to string_list
Mike Becker <universe@uap-core.de>
parents: 17
diff changeset
37 stringList->count = 0;
81
54515e5d60f5 fix ineffecient string list reallocation - fixes #637
Mike Becker <universe@uap-core.de>
parents: 57
diff changeset
38 stringList->capacity = 32;
101
0cb645809b1a code improvements that avoid warnings on higher warning levels
Mike Becker <universe@uap-core.de>
parents: 99
diff changeset
39 stringList->items = calloc(stringList->capacity, sizeof(char*));
94
9637e3efb8e7 add element destructor to string_list
Mike Becker <universe@uap-core.de>
parents: 82
diff changeset
40 stringList->free_item = do_not_free;
10
ecf787666f44 refactored sources
Mike Becker <universe@uap-core.de>
parents:
diff changeset
41
19
8bac9fd0629d generalized suffix_list to string_list
Mike Becker <universe@uap-core.de>
parents: 17
diff changeset
42 return stringList;
10
ecf787666f44 refactored sources
Mike Becker <universe@uap-core.de>
parents:
diff changeset
43 }
ecf787666f44 refactored sources
Mike Becker <universe@uap-core.de>
parents:
diff changeset
44
102
665b60727a89 improve code and increase const-correctness
Mike Becker <universe@uap-core.de>
parents: 101
diff changeset
45 void destroy_string_list(string_list *list) {
44
9574a181ec26 line sum per directory now displayed + directories without matching files are no longer displayed when -m is used + new buffering strategy replaces stream hack when -V is used
Mike Becker <universe@uap-core.de>
parents: 36
diff changeset
46 if (list) {
9574a181ec26 line sum per directory now displayed + directories without matching files are no longer displayed when -m is used + new buffering strategy replaces stream hack when -V is used
Mike Becker <universe@uap-core.de>
parents: 36
diff changeset
47 if (list->items) {
94
9637e3efb8e7 add element destructor to string_list
Mike Becker <universe@uap-core.de>
parents: 82
diff changeset
48 for (size_t i = 0 ; i < list->count ; i++) {
9637e3efb8e7 add element destructor to string_list
Mike Becker <universe@uap-core.de>
parents: 82
diff changeset
49 list->free_item(list->items[i]);
9637e3efb8e7 add element destructor to string_list
Mike Becker <universe@uap-core.de>
parents: 82
diff changeset
50 }
44
9574a181ec26 line sum per directory now displayed + directories without matching files are no longer displayed when -m is used + new buffering strategy replaces stream hack when -V is used
Mike Becker <universe@uap-core.de>
parents: 36
diff changeset
51 free(list->items);
9574a181ec26 line sum per directory now displayed + directories without matching files are no longer displayed when -m is used + new buffering strategy replaces stream hack when -V is used
Mike Becker <universe@uap-core.de>
parents: 36
diff changeset
52 }
9574a181ec26 line sum per directory now displayed + directories without matching files are no longer displayed when -m is used + new buffering strategy replaces stream hack when -V is used
Mike Becker <universe@uap-core.de>
parents: 36
diff changeset
53 free(list);
10
ecf787666f44 refactored sources
Mike Becker <universe@uap-core.de>
parents:
diff changeset
54 }
ecf787666f44 refactored sources
Mike Becker <universe@uap-core.de>
parents:
diff changeset
55 }
ecf787666f44 refactored sources
Mike Becker <universe@uap-core.de>
parents:
diff changeset
56
102
665b60727a89 improve code and increase const-correctness
Mike Becker <universe@uap-core.de>
parents: 101
diff changeset
57 void add_string(string_list *list, char *item) {
82
44b3332dfe03 greatly improve memory usage - fixes #634
Mike Becker <universe@uap-core.de>
parents: 81
diff changeset
58 assert(list != NULL);
44b3332dfe03 greatly improve memory usage - fixes #634
Mike Becker <universe@uap-core.de>
parents: 81
diff changeset
59 if (list->count < list->capacity) {
81
54515e5d60f5 fix ineffecient string list reallocation - fixes #637
Mike Becker <universe@uap-core.de>
parents: 57
diff changeset
60 list->items[list->count++] = item;
82
44b3332dfe03 greatly improve memory usage - fixes #634
Mike Becker <universe@uap-core.de>
parents: 81
diff changeset
61 return;
44
9574a181ec26 line sum per directory now displayed + directories without matching files are no longer displayed when -m is used + new buffering strategy replaces stream hack when -V is used
Mike Becker <universe@uap-core.de>
parents: 36
diff changeset
62 }
82
44b3332dfe03 greatly improve memory usage - fixes #634
Mike Becker <universe@uap-core.de>
parents: 81
diff changeset
63 size_t newCapacity = list->capacity * 2;
102
665b60727a89 improve code and increase const-correctness
Mike Becker <universe@uap-core.de>
parents: 101
diff changeset
64 char **reallocated_list =
82
44b3332dfe03 greatly improve memory usage - fixes #634
Mike Becker <universe@uap-core.de>
parents: 81
diff changeset
65 realloc(list->items, sizeof(char*) * newCapacity);
44b3332dfe03 greatly improve memory usage - fixes #634
Mike Becker <universe@uap-core.de>
parents: 81
diff changeset
66 assert(reallocated_list != NULL);
44b3332dfe03 greatly improve memory usage - fixes #634
Mike Becker <universe@uap-core.de>
parents: 81
diff changeset
67 list->capacity = newCapacity;
44b3332dfe03 greatly improve memory usage - fixes #634
Mike Becker <universe@uap-core.de>
parents: 81
diff changeset
68 list->items = reallocated_list;
44b3332dfe03 greatly improve memory usage - fixes #634
Mike Becker <universe@uap-core.de>
parents: 81
diff changeset
69 list->items[list->count++] = item;
44
9574a181ec26 line sum per directory now displayed + directories without matching files are no longer displayed when -m is used + new buffering strategy replaces stream hack when -V is used
Mike Becker <universe@uap-core.de>
parents: 36
diff changeset
70 }
9574a181ec26 line sum per directory now displayed + directories without matching files are no longer displayed when -m is used + new buffering strategy replaces stream hack when -V is used
Mike Becker <universe@uap-core.de>
parents: 36
diff changeset
71

mercurial