Changeset 33b8d024 in mainline for uspace/lib/hound/src/client.c

Timestamp:

2018-01-16T20:38:46Z (7 years ago)

Author:

Jiří Zárevúcky <zarevucky.jiri@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

2467b41

Parents:

d39c46e0

Message:

Remove const qualifier from the argument of free() and realloc(),
as well as in numerous other variables that hold ownership of memory.

By convention, a pointer that holds ownership is _never_ qualified by const.
This is reflected in the standard type signature of free() and realloc().
Allowing const pointers to hold ownership may seem superficially convenient,
but is actually quite confusing to experienced C programmers.

File:

• uspace/lib/hound/src/client.c (modified) (4 diffs)

uspace/lib/hound/src/client.c

@@ -76 +76 @@
         hound_sess_t *session;
         /** context name, reported to the daemon */
-        const char *name;
+        char *name;
         /** True if the instance is record context */
         bool record;
@@ -196 +196 @@
  */
 errno_t hound_context_get_available_targets(hound_context_t *hound,
-    const char ***names, size_t *count)
+    char ***names, size_t *count)
 {
         assert(hound);
@@ -213 +213 @@
  */
 errno_t hound_context_get_connected_targets(hound_context_t *hound,
-    const char ***names, size_t *count)
+    char ***names, size_t *count)
 {
         assert(hound);
@@ -237 +237 @@
         assert(target);
 
-        const char **tgt = NULL;
+        char **tgt = NULL;
         size_t count = 1;
         errno_t ret = EOK;