On 6/9/2020 12:59 PM, Qi Zhang wrote:
> This patch mainly does cleanups related to unnecessary braces.

Personally I think better to keep the braces when the block is more than single
line, otherwise you are mostly relying on indentation which is prone to error.

If you have strong opinion/need to remove them that is OK but if this is
optional my 5cents is to keep the braces.

> 
> A few other minor fixes are included which include fixing a typo,
> pulling up some lines, and fixing RCT.

I can't find the typo fix and I can't find othe change (fixing RCT), are they
still valid in this patch?

> 
> Signed-off-by: Tony Nguyen <anthony.l.ngu...@intel.com>
> Signed-off-by: Paul M. Stillwell Jr <paul.m.stillwell...@intel.com>
> Signed-off-by: Qi Zhang <qi.z.zh...@intel.com>

<...>

Reply via email to