On 06/05/2013 08:34 PM, Eric Blake wrote: Actually squashing this in first, to avoid the next failure of unused variables. Also, I really think we should ditch the code protected by: #elif HAVE_SYS_SYSCALL_H && defined(SYS_fallocate) as it is extremely unlikely these days to have a kernel with fallocate but a glibc that lacks the wrapper. Anyone on a system that out-of-date deserves to not be able to use this feature. diff --git a/src/util/virstoragefile.c b/src/util/virstoragefile.c index bf668c8..b985df4 100644 --- a/src/util/virstoragefile.c +++ b/src/util/virstoragefile.c @@ -1048,9 +1048,12 @@ virStorageFileResize(const char *path, { int fd = -1; int ret = -1; - int rc; - off_t offset = orig_capacity; - off_t len = capacity - orig_capacity; + int rc ATTRIBUTE_UNUSED; + off_t offset ATTRIBUTE_UNUSED; + off_t len ATTRIBUTE_UNUSED; + + offset = orig_capacity; + len = capacity - orig_capacity; if ((fd = open(path, O_RDWR)) < 0) { virReportSystemError(errno, _("Unable to open '%s'"), path); -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org