Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/lib/librefuse Use correct types for the fields of struct fus...



details:   https://anonhg.NetBSD.org/src/rev/ebba60daa524
branches:  trunk
changeset: 1026591:ebba60daa524
user:      pho <pho%NetBSD.org@localhost>
date:      Wed Dec 01 13:51:33 2021 +0000

description:
Use correct types for the fields of struct fuse_opt

This change shouldn't make any behavioral differences in reality, but
compilers these days dislike converting -1U to int32_t implicitly,
which is indeed understandable.

diffstat:

 lib/librefuse/fuse_opt.h   |  6 +++---
 lib/librefuse/refuse_opt.c |  4 ++--
 2 files changed, 5 insertions(+), 5 deletions(-)

diffs (38 lines):

diff -r 59245bab345a -r ebba60daa524 lib/librefuse/fuse_opt.h
--- a/lib/librefuse/fuse_opt.h  Tue Nov 30 23:58:10 2021 +0000
+++ b/lib/librefuse/fuse_opt.h  Wed Dec 01 13:51:33 2021 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: fuse_opt.h,v 1.7 2016/11/14 17:19:29 pho Exp $ */
+/*     $NetBSD: fuse_opt.h,v 1.8 2021/12/01 13:51:33 pho Exp $ */
 
 /*
  * Copyright (c) 2007 Alistair Crooks.  All rights reserved.
@@ -46,8 +46,8 @@
 
 struct fuse_opt {
        const char      *templ;
-       int32_t         offset;
-       int32_t         value;
+       unsigned long   offset;
+       int             value;
 };
 
 #define FUSE_OPT_KEY(templ, key) { templ, -1U, key }
diff -r 59245bab345a -r ebba60daa524 lib/librefuse/refuse_opt.c
--- a/lib/librefuse/refuse_opt.c        Tue Nov 30 23:58:10 2021 +0000
+++ b/lib/librefuse/refuse_opt.c        Wed Dec 01 13:51:33 2021 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: refuse_opt.c,v 1.19 2021/10/30 09:06:34 nia Exp $      */
+/*     $NetBSD: refuse_opt.c,v 1.20 2021/12/01 13:51:33 pho Exp $      */
 
 /*-
  * Copyright (c) 2007 Juan Romero Pardines.
@@ -286,7 +286,7 @@
                const struct fuse_opt* opt, int sep_idx, void* data,
                fuse_opt_proc_t proc, bool is_opt)
 {
-       if (opt->offset == -1) {
+       if (opt->offset == -1U) {
                /* The option description does not want any variables to be
                 * updated.*/
                if (call_proc(proc, data, arg, opt->value, outargs, is_opt) == -1)



Home | Main Index | Thread Index | Old Index