ok
Ross L Richardson(open...@rlr.id.au) on 2020.08.03 15:37:50 +1000:
> cppcheck reports that kv_inherit(), kv_log(), and print_time() are unused.
>
> The patch below deletes them.
>
> Ross
> --
>
> Index: httpd.c
> ===
> RCS file: /cvs/src/usr.sbin/httpd/httpd.c,v
> retrieving revision 1.69
> diff -u -p -r1.69 httpd.c
> --- httpd.c 30 Jul 2020 21:06:19 - 1.69
> +++ httpd.c 3 Aug 2020 05:35:09 -
> @@ -1059,50 +1059,6 @@ kv_free(struct kv *kv)
> }
>
> struct kv *
> -kv_inherit(struct kv *dst, struct kv *src)
> -{
> - memset(dst, 0, sizeof(*dst));
> - memcpy(dst, src, sizeof(*dst));
> - TAILQ_INIT(&dst->kv_children);
> -
> - if (src->kv_key != NULL) {
> - if ((dst->kv_key = strdup(src->kv_key)) == NULL) {
> - kv_free(dst);
> - return (NULL);
> - }
> - }
> - if (src->kv_value != NULL) {
> - if ((dst->kv_value = strdup(src->kv_value)) == NULL) {
> - kv_free(dst);
> - return (NULL);
> - }
> - }
> -
> - return (dst);
> -}
> -
> -int
> -kv_log(struct evbuffer *log, struct kv *kv)
> -{
> - char*msg;
> -
> - if (log == NULL)
> - return (0);
> - if (asprintf(&msg, " [%s%s%s]",
> - kv->kv_key == NULL ? "(unknown)" : kv->kv_key,
> - kv->kv_value == NULL ? "" : ": ",
> - kv->kv_value == NULL ? "" : kv->kv_value) == -1)
> - return (-1);
> - if (evbuffer_add(log, msg, strlen(msg)) == -1) {
> - free(msg);
> - return (-1);
> - }
> - free(msg);
> -
> - return (0);
> -}
> -
> -struct kv *
> kv_find(struct kvtree *keys, struct kv *kv)
> {
> struct kv *match;
> @@ -1270,22 +1226,6 @@ print_host(struct sockaddr_storage *ss,
> buf[0] = '\0';
> return (NULL);
> }
> - return (buf);
> -}
> -
> -const char *
> -print_time(struct timeval *a, struct timeval *b, char *buf, size_t len)
> -{
> - struct timeval tv;
> - unsigned long h, sec, min;
> -
> - timerclear(&tv);
> - timersub(a, b, &tv);
> - sec = tv.tv_sec % 60;
> - min = tv.tv_sec / 60 % 60;
> - h = tv.tv_sec / 60 / 60;
> -
> - snprintf(buf, len, "%.2lu:%.2lu:%.2lu", h, min, sec);
> return (buf);
> }
>
> Index: httpd.h
> ===
> RCS file: /cvs/src/usr.sbin/httpd/httpd.h,v
> retrieving revision 1.148
> diff -u -p -r1.148 httpd.h
> --- httpd.h 30 Jul 2020 21:06:19 - 1.148
> +++ httpd.h 3 Aug 2020 05:35:09 -
> @@ -731,8 +731,6 @@ void kv_delete(struct kvtree *, struct
> struct kv*kv_extend(struct kvtree *, struct kv *, char *);
> void kv_purge(struct kvtree *);
> void kv_free(struct kv *);
> -struct kv*kv_inherit(struct kv *, struct kv *);
> -int kv_log(struct evbuffer *, struct kv *);
> struct kv*kv_find(struct kvtree *, struct kv *);
> int kv_cmp(struct kv *, struct kv *);
> struct media_type
> @@ -751,7 +749,6 @@ struct auth *auth_add(struct serverauth
> struct auth *auth_byid(struct serverauth *, uint32_t);
> void auth_free(struct serverauth *, struct auth *);
> const char *print_host(struct sockaddr_storage *, char *, size_t);
> -const char *print_time(struct timeval *, struct timeval *, char *, size_t);
> const char *printb_flags(const uint32_t, const char *);
> void getmonotime(struct timeval *);
>
>