> --- a/utils.c > +++ b/utils.c > @@ -1476,11 +1475,11 @@ u64 parse_size(char *s) > return strtoull(s, NULL, 10) * mult; > } > > -int open_file_or_dir(const char *fname) > +int open_file_or_dir(const char *fname, DIR **dirstream) > { > int ret; > struct stat st; > - DIR *dirstream = NULL; > + *dirstream = NULL;
This is imho not needed, all call sites explicitly set dirstream to NULL which is more clear than hiding it inside a function as a sideefect, so I'll drop that line. > int fd; > > ret = stat(fname, &st); -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html