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