Libguestfs - Oct 2015 - Re: [PATCHv4 1/2] New API: vfs_minimum_size

On Tuesday 20 October 2015 18:45:55 Maxim Perevedentsev
wrote:
> +int64_t
> +do_vfs_minimum_size (const mountable_t *mountable)
> +{
> +  int64_t r;
> +
> +  /* How we set the label depends on the filesystem type. */
Just noticed now: this comment should be really fitting more the new
function... :)

-- 
Pino Toscano

On 10/20/2015 07:49 PM, Pino Toscano wrote:
> On Tuesday 20 October 2015 18:45:55 Maxim Perevedentsev wrote:
>> +int64_t
>> +do_vfs_minimum_size (const mountable_t *mountable)
>> +{
>> +  int64_t r;
>> +
>> +  /* How we set the label depends on the filesystem type. */
> Just noticed now: this comment should be really fitting more the new
> function... :)
Thanks. I'll fix it in final submission, after Richard succeeds in testing.

-- 
Your sincerely,
Maxim Perevedentsev

On Tue, Oct 20, 2015 at 08:46:14PM +0300, Maxim Perevedentsev
wrote:
> On 10/20/2015 07:49 PM, Pino Toscano wrote:
> >On Tuesday 20 October 2015 18:45:55 Maxim Perevedentsev wrote:
> >>+int64_t
> >>+do_vfs_minimum_size (const mountable_t *mountable)
> >>+{
> >>+  int64_t r;
> >>+
> >>+  /* How we set the label depends on the filesystem type. */
> >Just noticed now: this comment should be really fitting more the new
> >function... :)
> Thanks. I'll fix it in final submission, after Richard succeeds in
testing.
Don't worry about it - I pushed it already.

Thanks,

Rich.

-- 
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Read my programming and virtualization blog: http://rwmj.wordpress.com
virt-builder quickly builds VMs from scratch
http://libguestfs.org/virt-builder.1.html