Hello Christophe, thank you for the feedback.

On Mon, 2020-02-17 at 07:31 +0100, Christophe Leroy wrote:
> >             if (old.oprofile_cpu_type != NULL) {
> >                     t->oprofile_cpu_type = old.oprofile_cpu_type;
> > -                   t->oprofile_type = old.oprofile_type;
> >             }
> 
> The action being reduced to a single line, the { } should be removed.
> 
> Christophe

I intentionally let it this way because I just reviewed a patch that
will add more itens here, and should be merged before this one.

This will avoid conflicts.

Best regards,
Leonardo Bras

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to