Re: [PATCH v3 04/14] pyjailhouse: sysfs_parser: minor stylistic fixups

[Ralf Ramsauer]

On 10/8/19 6:01 PM, Jan Kiszka wrote:
> On 08.10.19 17:56, Ralf Ramsauer wrote:
>>>> @@ -950,8 +947,8 @@ class IOMemRegionTree:
>>>>                  continue
>>>>  
>>>>              # if the tree continues recurse further down ...
>>>> -            if (len(tree.children) > 0):
>>>> -                (temp_regions, temp_dmar_regions) = \
>>>> +            if len(tree.children) > 0:
>>>> +                temp_regions, temp_dmar_regions = \
>>
>> Will re-add brackets here, and in all other patches.
>>
> 
> Why that?

Erm, because you commented "Style" for those cases in all other patches?
I'm referring to the second line of the hunk, not the 'if'-line.

  Ralf

-- 
You received this message because you are subscribed to the Google Groups 
"Jailhouse" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to jailhouse-dev+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/jailhouse-dev/144f50bb-ccfa-403a-333c-da36fe55effb%40oth-regensburg.de.