On Tue, 2015-06-30 at 10:25 +0100, Ian Jackson wrote:
> Ian Campbell writes ("Re: [OSSTEST PATCH 05/13] Reporting: report_run_getinfo
> produces ColourAttr"):
> > On Mon, 2015-06-29 at 18:14 +0100, Ian Jackson wrote:
> ...
> > > - return { Summary => $summary, Colour => $colour };
> > > + return {
> > > + Summary => $summary,
> > > + ColourAttr => " bgcolor=\"$colour\"",
> >
> > FWIW all the uses have a space before there $ri->{ColourAttr}, so
> > strictly you don't need it here, I think.
>
> Ah yes.
>
> > Nonetheless:
> >
> > Acked-by: Ian Campbell <ian.campb...@citrix.com>
>
> Thanks. I will remove that space and keep your ack.
Sure. I spotted another later in the series which I commented on too.
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
