> On Wed, Sep 7, 2016 at 4:57 PM, Christophe Fergeau < cferg...@redhat.com > > wrote:
> > On Tue, Sep 06, 2016 at 05:29:22PM +0300, Sameeh Jubran wrote: > > > > It is the recommended minimum, however Windows can handle low resolution > > > > just fine. We have tested that, > > > > even the old driver behaves like this. I see no reason for making any > > > > restriction at all. > > > Can you please add this kind of reasoning/explanation to the very terse > > > commit log which is there currently ? > > Sure > > Thanks, > > > Christophe > This patch is just changing the "Add arbitrary resolution and monitors_config Escape" patch which is not in master, so would be better to squash there. Also the "Add arbitrary resolution and monitors_config Escape" already has a rationale for this patch that is: "Windows can handle low resolution just fine, the old (XPDM) driver behaves like this too. There is no reason for making any restriction at all." (actually this rationale apply the the wrong patch!) Frediano
_______________________________________________ Spice-devel mailing list Spice-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/spice-devel
