[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