[PATCH 7/7] wined3d: Move A4L4 conversion to the formats table.

[Sven Baars]

Greg Geldorp schreef:
>> From: Ge van Geldorp
>>     
>>> From: Roderick Colenbrander 
>>>
>>> I'm not sure how the bot works but it looks like it concatenated some
>>> of my patches together and then applied to them. I have no idea to
>>> what Wine version it was since it applies fine on Wine git here. It
>>> could be that I'm doing something wrong but I don't see where.
>>>       
>> No, this is the bot messing up. For some reason I don't understand yet
>> it included one of your earlier patches ("[PATCH 08/13] d3d9: Add an
>> initial ColorFill regression test.") as the first of this series. That
>> threw everything off.
>>
>> I'll disable emailing to wine-devel until I've sorted this out.
>>     
>
> Ok, I understand what happened now... This is the sequence of patches you
> submitted, in the order they arrived in my mail queue (oldest first):
>
> 1) [PATCH 4/7] wined3d: Move D15S1 over to the formats table.
> 2) [PATCH 1/7] wined3d: Move L6V5U5 conversion to the formats table.
> 3) [PATCH 5/7] wined3d: Move R32G32F convertion to the formats table.
> 4) [PATCH 3/7] wined3d: Move D24X4S4 to the formats table.
> 5) [PATCH 6/7] wined3d: Move G16R16/R16G16F conversion to the formats table.
> 6) 1/3 d3d9: Add an initial ColorFill regression test.
> 7) [PATCH 2/7] wined3d: Move D24FS8 to formats table.
> 8) [PATCH 7/7] wined3d: Move A4L4 conversion to the formats table.
>
> Note the out-of-sequence arrival of number 6). The bot is not smart enough
> to handle two series from the same author arriving interleaved. It will see
> everything as one big series until all parts of a series have arrived. In
> this case, the patch attached to message 6) overwrote the patch attached
> to message 2) (both have part number 1).
>
> Not sure how to handle this, except by asking people to leave some time
> before sending another series.
>
> Re-enabling emailing to wine-devel since this is working as designed.
>
> Ge.
>   
Hey Ge,

you could, instead of only checking for the author, also try to check 
for the maximum amount of patches. If you find patch [Patch 1/7], I 
assume you currently wait for all 7 patches. Now if a patch 1/3 comes in 
between, you'll see that the maximum is different, in this case 3, so 
you'll know that it's a different patch-set.

In case the maximum is also the same, it's also difficult for a human to 
see what patch belongs to what patch-set, so that should never happen in 
the first place. Unless AJ is a psychic of course.

Sven

P.S. Awesome work on the testbot.