On Tue, 2015-03-17 at 15:44 +0000, Wei Liu wrote:
> On Mon, Mar 16, 2015 at 10:06:17AM +0000, PRAMOD DEVENDRA wrote:
> > From: Pramod Devendra <pramod.deven...@citrix.com>
> > 
> > Signed-off-by: Pramod Devendra <pramod.deven...@citrix.com>
> > CC: Ian Jackson <ian.jack...@eu.citrix.com>
> > CC: Stefano Stabellini <stefano.stabell...@eu.citrix.com>
> > CC: Ian Campbell <ian.campb...@citrix.com>
> > CC: Wei Liu <wei.l...@citrix.com>
> 
> Acked-by: Wei Liu <wei.l...@citrix.com>

Applied.

> >      resstr = entry->policy[flag->reg - 1];
> > -    if (resstr == NULL) {
> > -        resstr = strdup("xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx");
> > -    }
> 
> Minor nit. I would prefer "resstr = " be grouped with the code you
> moved. No need to resend though.

Do you mean the restr = entry->... bit?



_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Reply via email to