Re: [Xen-devel] [PATCH 1/3] page-alloc: Rename the first_node local variable

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 25.06.19 at 16:43, <andrew.cooper3@xxxxxxxxxx> wrote:
> first_node is the name of a local variable, and part of the nodemask API.  The
> only reason this compiles is because the nodemask API is implemented as a
> macro rather than an inline function.
> 
> It is confusing to read, and breaks when the nodemask API is cleaned up.
> Rename the local variable to just 'first' which is still clear in context.
> 
> No functional change.
> 
> Signed-off-by: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>

Acked-by: Jan Beulich <jbeulich@xxxxxxxx>



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel