Changeset 8591b31 in mainline for uspace/lib

Timestamp:

2018-09-20T16:13:43Z (7 years ago)

Author:

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

Branches:

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

Children:

cbc3587

Parents:

9a5ae815

Message:

Fix most clang issues

Build error in futex.c remains because the proper solution of using
type _Atomic(fibril_t *) can't be parsed by sycek.

Location:

uspace/lib

Files:

• c/generic/private/futex.h (modified) (1 diff)
• c/test/string.c (modified) (1 diff)
• virtio/virtio.c (modified) (1 diff)

uspace/lib/c/generic/private/futex.h

@@ -41 +41 @@
 #include <libc.h>
 #include <time.h>
+#include <fibril.h>
 
 typedef struct futex {
         volatile atomic_int val;
 #ifdef CONFIG_DEBUG_FUTEX
-        void *owner;
+        // FIXME: Should be _Atomic(fibril_t *)
+        fibril_t *owner;
 #endif
 } futex_t;

uspace/lib/c/test/string.c

@@ -33 +33 @@
 #include <pcut/pcut.h>
 
+#ifndef __clang__
 #pragma GCC diagnostic ignored "-Wstringop-truncation"
 #pragma GCC diagnostic ignored "-Wstringop-overflow"
+#endif
 
 PCUT_INIT;

uspace/lib/virtio/virtio.c

@@ -127 +127 @@
         for (unsigned i = 0; i < size; i++) {
                 virtio_virtq_desc_set(vdev, num, i, 0, 0,
-                    VIRTQ_DESC_F_NEXT, (i + 1 == size) ? -1U : i + 1);
+                    VIRTQ_DESC_F_NEXT, (i + 1 == size) ? 0xffffu : i + 1);
         }
         *head = 0;