[Spice-devel] [PATCH win-vdagent] Provide support for Windows CCD API

[Christophe Fergeau]

On Wed, Aug 03, 2016 at 04:04:50PM +0300, Dmitry Fleytman wrote:
> 
> > On 3 Aug 2016, at 11:57 AM, Frediano Ziglio <fziglio at redhat.com> wrote:
> > 
> >> 
> >> On Wed, Aug 03, 2016 at 11:35:56AM +0300, Sameeh Jubran wrote:
> >>> Hi,
> >>> 
> >>> We've already looked into those patches (Javier's patches) and tested them.
> >> 
> >> Ah ok, I must have missed your answer to Javier's series.
> >> 
> >>> We found that on the contrary to our patches,
> >>> multi-monitor support and arbitrary resolution do not work with the patches
> >>> posted by Javier. Even though arbitrary resolution
> >>> resolution is implemented in Javier's patches, it is not fully arbitrary.
> >> 
> >> And full arbitrary resolution would be too complicated to add on top of
> >> Javier's series, hence the decision to go with an entirely different
> >> series? Sorry for the basic questions, I'm really not familiar with
> >> Windows API :)
> >> 
> >> Christophe
> >> 
> > 
> > One good thing about Javier's patches, they were small.
> > The current patch is too big and should be split.
> 
> Please note that both series (this and Javier’s) are based on the same approach,
> so they will be of similar size after Javier’s patches amended with missing features.

But we would have smaller incremental patches leading to the end result,
rather than one big patch doing everything at once, wouldn't we?

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/spice-devel/attachments/20160803/7f267415/attachment.sig>