Re: [PATCH spice-gtk] Notify about existence of monitor for all display channels

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxx> 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.

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]