[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-ppc] [RFC for-4.1 20/25] target/ppc: Style fixes for translate
From: |
Cédric Le Goater |
Subject: |
Re: [Qemu-ppc] [RFC for-4.1 20/25] target/ppc: Style fixes for translate_init.inc.c |
Date: |
Mon, 25 Mar 2019 08:55:29 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.4.0 |
On 3/25/19 7:52 AM, David Gibson wrote:
> On Mon, Mar 25, 2019 at 07:39:32AM +0100, Cédric Le Goater wrote:
>> On 3/22/19 1:15 AM, David Gibson wrote:
>>> Signed-off-by: David Gibson <address@hidden>
>>
>> There is a printf() we could change below. Anyhow,
>
> What did you have in mind?
[ ... ]
This printf could be changed to a error_setg() but it should be in
another patch.
C.
>>> @@ -9385,8 +9428,9 @@ static int test_opcode_table(opc_handler_t **table,
>>> int len)
>>>
>>> static void fix_opcode_tables(opc_handler_t **ppc_opcodes)
>>> {
>>> - if (test_opcode_table(ppc_opcodes, PPC_CPU_OPCODES_LEN) == 0)
>>> + if (test_opcode_table(ppc_opcodes, PPC_CPU_OPCODES_LEN) == 0) {
>>> printf("*** WARNING: no opcode defined !\n");
>>> + }
>>> }