Module Name:    src
Committed By:   christos
Date:           Sun Nov 26 03:51:45 UTC 2017

Modified Files:
        src/sbin/mount_qemufwcfg: virtdir.c virtdir.h

Log Message:
more cleanups, const


To generate a diff of this commit:
cvs rdiff -u -r1.2 -r1.3 src/sbin/mount_qemufwcfg/virtdir.c \
    src/sbin/mount_qemufwcfg/virtdir.h

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sbin/mount_qemufwcfg/virtdir.c
diff -u src/sbin/mount_qemufwcfg/virtdir.c:1.2 src/sbin/mount_qemufwcfg/virtdir.c:1.3
--- src/sbin/mount_qemufwcfg/virtdir.c:1.2	Sat Nov 25 22:06:24 2017
+++ src/sbin/mount_qemufwcfg/virtdir.c	Sat Nov 25 22:51:45 2017
@@ -1,4 +1,4 @@
-/* $NetBSD: virtdir.c,v 1.2 2017/11/26 03:06:24 christos Exp $ */
+/* $NetBSD: virtdir.c,v 1.3 2017/11/26 03:51:45 christos Exp $ */
 
 /*
  * Copyright © 2007 Alistair Crooks.  All rights reserved.
@@ -44,8 +44,8 @@
 static int
 compare(const void *vp1, const void *vp2)
 {
-	const virt_dirent_t *tp1 = (const virt_dirent_t *) vp1;
-	const virt_dirent_t *tp2 = (const virt_dirent_t *) vp2;
+	const virt_dirent_t *tp1 = vp1;
+	const virt_dirent_t *tp2 = vp2;
 
 	return strcmp(tp1->name, tp2->name);
 }
@@ -60,7 +60,7 @@ mkdirs(virtdir_t *tp, const char *path, 
 
 	(void) strlcpy(name, path, sizeof(name));
 	for (slash = name + 1 ; (slash = strchr(slash + 1, '/')) != NULL ; ) {
-		*slash = 0x0;
+		*slash = '\0';
 		if ((ep = virtdir_find(tp, name, strlen(name))) == NULL) {
 			virtdir_add(tp, name, strlen(name), 'd', NULL, 0, 0);
 		}
@@ -101,7 +101,8 @@ normalise(const char *name, size_t namel
 
 /* initialise the tree */
 int
-virtdir_init(virtdir_t *tp, const char *rootdir, struct stat *d, struct stat *f, struct stat *l)
+virtdir_init(virtdir_t *tp, const char *rootdir, const struct stat *d,
+    const struct stat *f, const struct stat *l)
 {
 	tp->dir = *d;
 	tp->dir.st_mode = S_IFDIR | 0755;
@@ -120,7 +121,8 @@ virtdir_init(virtdir_t *tp, const char *
 
 /* add an entry to the tree */
 int
-virtdir_add(virtdir_t *tp, const char *name, size_t size, uint8_t type, const char *tgt, size_t tgtlen, uint16_t select)
+virtdir_add(virtdir_t *tp, const char *name, size_t size, uint8_t type,
+    const char *tgt, size_t tgtlen, uint16_t select)
 {
 	char		path[MAXPATHLEN];
 	size_t		pathlen;
@@ -131,13 +133,11 @@ virtdir_add(virtdir_t *tp, const char *n
 		return 0;
 	}
 	if (tp->c == tp->size || tp->size == 0) {
-		tp->size += 10,
+		tp->size += 10;
 		tp->v = erealloc(tp->v, tp->size * sizeof(*tp->v));
 	}
 	tp->v[tp->c].namelen = pathlen;
-	if ((tp->v[tp->c].name = estrndup(path, pathlen)) == NULL) {
-		return 0;
-	}
+	tp->v[tp->c].name = estrndup(path, pathlen);
 	tp->v[tp->c].d_name = strrchr(tp->v[tp->c].name, '/') + 1;
 	tp->v[tp->c].type = type;
 	tp->v[tp->c].ino = (ino_t) random() & 0xfffff;
@@ -159,7 +159,7 @@ virtdir_find(virtdir_t *tp, const char *
 	virt_dirent_t	e;
 	char		path[MAXPATHLEN];
 
-	(void) memset(&e, 0x0, sizeof(e));
+	(void) memset(&e, 0, sizeof(e));
 	e.namelen = normalise(name, namelen, path, sizeof(path));
 	e.name = path;
 	return bsearch(&e, tp->v, tp->c, sizeof(tp->v[0]), compare);
@@ -167,7 +167,7 @@ virtdir_find(virtdir_t *tp, const char *
 
 /* return the virtual offset in the tree */
 off_t
-virtdir_offset(virtdir_t *tp, virt_dirent_t *dp)
+virtdir_offset(const virtdir_t *tp, const virt_dirent_t *dp)
 {
 	return dp - tp->v;
 }
@@ -194,7 +194,7 @@ readvirtdir(VIRTDIR *dirp)
 {
 	char	*from;
 
-	for ( ; dirp->i < dirp->tp->c ; dirp->i++) {
+	for ( ; dirp->i < dirp->tp->c; dirp->i++) {
 		from = (strcmp(dirp->dirname, "/") == 0) ?
 		    &dirp->tp->v[dirp->i].name[1] :
 		    &dirp->tp->v[dirp->i].name[dirp->dirnamelen + 1];
Index: src/sbin/mount_qemufwcfg/virtdir.h
diff -u src/sbin/mount_qemufwcfg/virtdir.h:1.2 src/sbin/mount_qemufwcfg/virtdir.h:1.3
--- src/sbin/mount_qemufwcfg/virtdir.h:1.2	Sat Nov 25 22:06:24 2017
+++ src/sbin/mount_qemufwcfg/virtdir.h	Sat Nov 25 22:51:45 2017
@@ -1,4 +1,4 @@
-/* $NetBSD: virtdir.h,v 1.2 2017/11/26 03:06:24 christos Exp $ */
+/* $NetBSD: virtdir.h,v 1.3 2017/11/26 03:51:45 christos Exp $ */
 
 /*
  * Copyright © 2007 Alistair Crooks.  All rights reserved.
@@ -66,14 +66,16 @@ typedef struct VIRTDIR {
 	size_t		 i;		/* current offset in dir tree */
 } VIRTDIR;
 
-int virtdir_init(virtdir_t *, const char *, struct stat *, struct stat *, struct stat *);
-int virtdir_add(virtdir_t *, const char *, size_t, uint8_t, const char *, size_t, uint16_t);
+int virtdir_init(virtdir_t *, const char *, const struct stat *,
+    const struct stat *, const struct stat *);
+int virtdir_add(virtdir_t *, const char *, size_t, uint8_t, const char *,
+    size_t, uint16_t);
 virt_dirent_t *virtdir_find(virtdir_t *, const char *, size_t);
 
 VIRTDIR *openvirtdir(virtdir_t *, const char *);
 virt_dirent_t *readvirtdir(VIRTDIR *);
 void closevirtdir(VIRTDIR *);
 
-off_t virtdir_offset(virtdir_t *, virt_dirent_t *);
+off_t virtdir_offset(const virtdir_t *, const virt_dirent_t *);
 
 #endif

Reply via email to