Otavio Salvador <[EMAIL PROTECTED]> wrote:
> "Debarshi 'Rishi' Ray" <[EMAIL PROTECTED]> writes:
>
>> Some more printf-like functions replaced by fputs, puts and putchar. I
>> have tried to avoid any re-formatting than what was absolutely
>> necessary to preserve the formatting of the code after making the
>> changes.
>>
>> Comments?
>
> Some comments.
>
>> --- a/libparted/fs/ext2/ext2_inode_relocator.c
>> +++ b/libparted/fs/ext2/ext2_inode_relocator.c
>> @@ -309,7 +309,7 @@ static int doscan(struct ext2_fs *fs, struct
>> ext2_inode_relocator_state *state)
>> }
>>
>> if (fs->opt_verbose)
>> - fprintf(stderr, "\n");
>> + fputc ('\n', stderr);
>
> I don't know if we should or not also change the indentation on the
> changed lines. I'm ok with it but I would like to hear from the others
> too.
>
> There're a lot of indentation changes on the patch.
I noticed, but didn't say anything about this or other recent patches
on the presumption that we'll have to do some global reformatting
eventually anyhow. The current situation is untenable.
_______________________________________________
parted-devel mailing list
[email protected]
http://lists.alioth.debian.org/mailman/listinfo/parted-devel
