Re: [v6 01/13] d3d9: Call wined3d_device_copy_sub_resource_region() instead of wined3d_device_update_surface() in d3d9_device_UpdateSurface().

[Riccardo Bortolato]

Hi,

what can I do to improve patches 1 to 6 in order to get them accepted?

I will be on IRC for quicker discussion.

Ciao,
Riccardo

2015-11-05 17:15 GMT+01:00 Marvin <testbot(a)winehq.org>:
> Thank you for your contribution to Wine!
>
> This is an automated notification to let you know that your patch has
> been reviewed and its status set to "Assigned".
>
> This means that the patch has been assigned to a reviewer. If the
> reviewer signs off on the patch, it will be committed, otherwise
> you'll receive advice on how to improve it. If you don't hear anything
> after a couple of weeks, please resend the patch.

[v6 01/13] d3d9: Call wined3d_device_copy_sub_resource_region() instead of wined3d_device_update_surface() in d3d9_device_UpdateSurface().

[Riccardo Bortolato]

Hi,

I submitted v7 of this serie (based on some hint by Józef Kucia).

I would appreciate getting any kind of feedback like "we don't have
time for this", "this patch is wrong and will never be accepted" or
"this needs more testing in wine-staging" because since we're at this
point I wouldn't like wasting my cleanup work (currently a ~35 patches
serie on github if anyone is interested).

Ciao,
Riccardo

2015-11-09 10:52 GMT+01:00 Riccardo Bortolato <rikyz619(a)gmail.com>:
> Hi,
>
> what can I do to improve patches 1 to 6 in order to get them accepted?
>
> I will be on IRC for quicker discussion.
>
> Ciao,
> Riccardo
>
> 2015-11-05 17:15 GMT+01:00 Marvin <testbot(a)winehq.org>:
> > Thank you for your contribution to Wine!
> >
> > This is an automated notification to let you know that your patch has
> > been reviewed and its status set to "Assigned".
> >
> > This means that the patch has been assigned to a reviewer. If the
> > reviewer signs off on the patch, it will be committed, otherwise
> > you'll receive advice on how to improve it. If you don't hear anything
> > after a couple of weeks, please resend the patch.

[v6 01/13] d3d9: Call wined3d_device_copy_sub_resource_region() instead of wined3d_device_update_surface() in d3d9_device_UpdateSurface().

[Henri Verbeet]

On 10 November 2015 at 14:22, Riccardo Bortolato <rikyz619(a)gmail.com> wrote:
> I submitted v7 of this serie (based on some hint by Józef Kucia).
>
> I would appreciate getting any kind of feedback like "we don't have
> time for this", "this patch is wrong and will never be accepted" or
> "this needs more testing in wine-staging" because since we're at this
> point I wouldn't like wasting my cleanup work (currently a ~35 patches
> serie on github if anyone is interested).

There were some issues with some of those patches (not necessarily
huge ones), but I've been a bit busy to write replies to them.
Hopefully I'll get to it this week.