]> granicus.if.org Git - libevent/commitdiff
Fixes some new warnings under clang-15
authorAzat Khuzhin <azat@libevent.org>
Fri, 27 Jan 2023 07:57:33 +0000 (08:57 +0100)
committerAzat Khuzhin <azat@libevent.org>
Fri, 27 Jan 2023 07:58:51 +0000 (08:58 +0100)
- -Wdeprecated-non-prototype

  /src/le/libevent/strlcpy.c:48:1: warning: a function definition without a prototype is deprecated in all versions of C and is not supported in C2x [-Wdeprecated-non-prototype]
  event_strlcpy_(dst, src, siz)

- -Wstrict-prototypes

  /src/le/libevent/evthread.c:82:70: warning: a function declaration without a prototype is deprecated in all versions of C [-Wstrict-prototypes]
  struct evthread_condition_callbacks *evthread_get_condition_callbacks()

- -Wunused-but-set-variable

  /src/le/libevent/test/regress_buffer.c:130:6: warning: variable 'n' set but not used [-Wunused-but-set-variable]
          int n = 0;
                                                                     ^

evthread.c
strlcpy.c
test/regress_buffer.c

index 3eac594d64dc8967f599f872568c1b425d9cde67..c2da914da1fd71fa1eae892f1b50678ffac386e3 100644 (file)
@@ -74,12 +74,12 @@ evthread_set_id_callback(unsigned long (*id_fn)(void))
        evthread_id_fn_ = id_fn;
 }
 
-struct evthread_lock_callbacks *evthread_get_lock_callbacks()
+struct evthread_lock_callbacks *evthread_get_lock_callbacks(void)
 {
        return evthread_lock_debugging_enabled_
            ? &original_lock_fns_ : &evthread_lock_fns_;
 }
-struct evthread_condition_callbacks *evthread_get_condition_callbacks()
+struct evthread_condition_callbacks *evthread_get_condition_callbacks(void)
 {
        return evthread_lock_debugging_enabled_
            ? &original_cond_fns_ : &evthread_cond_fns_;
index 3876475f5a6fa4d6e8aff897daee2f0b491fde63..04c74298dcad7772147edabed9aa56525c31dea0 100644 (file)
--- a/strlcpy.c
+++ b/strlcpy.c
@@ -44,11 +44,7 @@ static char *rcsid = "$OpenBSD: strlcpy.c,v 1.5 2001/05/13 15:40:16 deraadt Exp
  * will be copied.  Always NUL terminates (unless siz == 0).
  * Returns strlen(src); if retval >= siz, truncation occurred.
  */
-size_t
-event_strlcpy_(dst, src, siz)
-       char *dst;
-       const char *src;
-       size_t siz;
+size_t event_strlcpy_(char *dst, const char *src, size_t siz)
 {
        register char *d = dst;
        register const char *s = src;
index 5683810e263a83d77146c5596f63001dcaf2505e..b0a9e0c162f53e6d4ceaa2ae3f1fdae950deae9a 100644 (file)
@@ -127,19 +127,16 @@ evbuffer_get_waste(struct evbuffer *buf, size_t *allocatedp, size_t *wastedp, si
 {
        struct evbuffer_chain *chain;
        size_t a, w, u;
-       int n = 0;
        u = a = w = 0;
 
        chain = buf->first;
        /* skip empty at start */
        while (chain && chain->off==0) {
-               ++n;
                a += chain->buffer_len;
                chain = chain->next;
        }
        /* first nonempty chain: stuff at the end only is wasted. */
        if (chain) {
-               ++n;
                a += chain->buffer_len;
                u += chain->off;
                if (chain->next && chain->next->off)
@@ -148,7 +145,6 @@ evbuffer_get_waste(struct evbuffer *buf, size_t *allocatedp, size_t *wastedp, si
        }
        /* subsequent nonempty chains */
        while (chain && chain->off) {
-               ++n;
                a += chain->buffer_len;
                w += (size_t)chain->misalign;
                u += chain->off;
@@ -158,7 +154,6 @@ evbuffer_get_waste(struct evbuffer *buf, size_t *allocatedp, size_t *wastedp, si
        }
        /* subsequent empty chains */
        while (chain) {
-               ++n;
                a += chain->buffer_len;
        }
        *allocatedp = a;