[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/2] Block: Cleanup vvfat.c to remove dead code.
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH 1/2] Block: Cleanup vvfat.c to remove dead code. |
Date: |
Wed, 27 Apr 2016 16:20:49 +0100 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Fri, Apr 15, 2016 at 08:32:53AM +0530, Prerna Saxena wrote:
> Commit 43dc2a64 replaced assert() with abort(), but didnt remove statements
> that followed these calls. So current code still has return values set after
> a call to abort(). Such statements will never execute and need to be cleaned
> up.
>
> Signed-off-by: Prerna Saxena <address@hidden>
> ---
> block/vvfat.c | 17 +++--------------
> 1 file changed, 3 insertions(+), 14 deletions(-)
>
> diff --git a/block/vvfat.c b/block/vvfat.c
> index 6b85314..ffe739b 100644
> --- a/block/vvfat.c
> +++ b/block/vvfat.c
> @@ -1747,8 +1747,7 @@ static uint32_t
> get_cluster_count_for_direntry(BDRVVVFATState* s,
> schedule_new_file(s, g_strdup(path), cluster_num);
> else {
> abort();
> - return 0;
> - }
> + }
The following change breaks indentation:
^Ielse {
...
-^I}
+^I }
Since the else statement only has a single tab character it's wrong to
have mixed tabs and spaces for the closing curly.
Please resend this patch with tab-only indentation. The diff hunks
below sometimes introduce spaces.
signature.asc
Description: PGP signature