Re: [Qemu-ppc] [RFC for-4.1 07/25] target/ppc: Style fixes for dfp_helper.c

[Cédric Le Goater]

On 3/22/19 1:15 AM, David Gibson wrote:
> Signed-off-by: David Gibson <address@hidden>



Reviewed-by: Cédric Le Goater <address@hidden>

Thanks,

C.

> ---
>  target/ppc/dfp_helper.c | 14 +++++++-------
>  1 file changed, 7 insertions(+), 7 deletions(-)
> 
> diff --git a/target/ppc/dfp_helper.c b/target/ppc/dfp_helper.c
> index 9164fe701b..ddab65c7ba 100644
> --- a/target/ppc/dfp_helper.c
> +++ b/target/ppc/dfp_helper.c
> @@ -1104,19 +1104,19 @@ void helper_##op(CPUPPCState *env, uint64_t *t, 
> uint64_t *b, uint32_t s)     \
>          }                                                                    
> \
>          }                                                                    
> \
>                                                                               
> \
> -    while (offset < (size)/4) {                                              
> \
> +    while (offset < (size) / 4) {                                            
> \
>          n++;                                                                 
> \
> -        digits[(size)/4-n] = dfp_get_bcd_digit_##size(dfp.b64, offset++);    
> \
> -        if (digits[(size)/4-n] > 10) {                                       
> \
> +        digits[(size) / 4 - n] = dfp_get_bcd_digit_##size(dfp.b64, 
> offset++); \
> +        if (digits[(size) / 4 - n] > 10) {                                   
> \
>              dfp_set_FPSCR_flag(&dfp, FP_VX | FP_VXCVI, FPSCR_VE);            
> \
>              return;                                                          
> \
>          } else {                                                             
> \
> -            nonzero |= (digits[(size)/4-n] > 0);                             
> \
> +            nonzero |= (digits[(size) / 4 - n] > 0);                         
> \
>          }                                                                    
> \
>      }                                                                        
> \
>                                                                               
> \
>      if (nonzero) {                                                           
> \
> -        decNumberSetBCD(&dfp.t, digits+((size)/4)-n, n);                     
> \
> +        decNumberSetBCD(&dfp.t, digits+((size) / 4) - n, n);                 
> \
>      }                                                                        
> \
>                                                                               
> \
>      if (s && sgn)  {                                                         
> \
> @@ -1170,13 +1170,13 @@ DFP_HELPER_XEX(dxexq, 128)
>  static void dfp_set_raw_exp_64(uint64_t *t, uint64_t raw)
>  {
>      *t &= 0x8003ffffffffffffULL;
> -    *t |= (raw << (63-13));
> +    *t |= (raw << (63 - 13));
>  }
>  
>  static void dfp_set_raw_exp_128(uint64_t *t, uint64_t raw)
>  {
>      t[HI_IDX] &= 0x80003fffffffffffULL;
> -    t[HI_IDX] |= (raw << (63-17));
> +    t[HI_IDX] |= (raw << (63 - 17));
>  }
>  
>  #define DFP_HELPER_IEX(op, size)                                          \
>