[Spice-devel] [PATCH spice-gtk] Notify about existence of monitor for all display channels

Jonathon Jongsma jjongsma at redhat.com
Fri Jul 31 07:27:45 PDT 2015


On Fri, 2015-07-31 at 11:03 +0200, Pavel Grunt wrote:
> On Fri, 2015-07-31 at 10:53 +0200, Marc-André Lureau wrote:
> > Hi Pavel
> > 
> > On Fri, Jul 31, 2015 at 10:36 AM, Pavel Grunt <pgrunt at redhat.com> wrote:
> > > > I'm comfortable ACKing this patch, but I'd like to have the comment above
> > > > this line of code changed or removed. It's a bit confusing.
> > > 
> > > Ok, I will change it to "notify of existence of this monitor"
> > > 
> > 
> > I am quite sure he meant to remove this comment with this patch:
> >      /* if we are not using monitors config, notify of existence of
> >         this monitor */
> > 
> yeah, and I wanted to change it to /* notify of existence of this monitor */
> but if you preffer removing it at all, I will do it
> 
> cheers,
> Pavel

I think either solution is OK.



More information about the Spice-devel mailing list