qemu-trivial
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-trivial] [Qemu-devel] [PATCH] target-mips: Clean up switch fal


From: Peter Maydell
Subject: Re: [Qemu-trivial] [Qemu-devel] [PATCH] target-mips: Clean up switch fall through after commit fecd264
Date: Tue, 20 Jan 2015 18:35:38 +0000

On 20 January 2015 at 18:16, Maciej W. Rozycki <address@hidden> wrote:
> On Tue, 20 Jan 2015, Markus Armbruster wrote:
>
>> >> diff --git a/target-mips/translate.c b/target-mips/translate.c
>> >> index e9d86b2..8abc12b 100644
>> >> --- a/target-mips/translate.c
>> >> +++ b/target-mips/translate.c
>> >> @@ -18729,6 +18729,7 @@ static void decode_opc(CPUMIPSState *env, 
>> >> DisasContext *ctx)
>> >>      case OPC_SWL:
>> >>      case OPC_SWR:
>> >>          check_insn_opc_removed(ctx, ISA_MIPS32R6);
>> >> +         /* fall through */
>> >
>> > Indent here seems to be out by one? The others look OK.
>>
>> Sorry about that.  Fix up on commit, or would you like a respin?
>
>  It looks to me like this whole file requires reindentation, perhaps a
> mechanical update like that would be better.  There are many lines with 9
> leading spaces (`wc -l' tells me 20), that I deliberately left as they are
> with my recent patches in this area so as not to obfuscate semantic
> changes.  There may be some more issues.

In this particular case, this part of the file is fine and the
problem is simply that this patch as it stands introduces a single
line (the one above) that's not indented correctly. The only fix
required is to delete one space in the line added by the patch
(thus avoiding introducing any new misindented lines).

In the general case, we tend to not do whole-file reindentation,
because it breaks 'git blame' and similar tools. It's not an
outright ban, though -- I guess it comes down to a maintainer
judgement call whether they think the benefit outweighs the cost
for a particular bit of code.

-- PMM



reply via email to

[Prev in Thread] Current Thread [Next in Thread]