<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">2013/6/25 Henri Verbeet <span dir="ltr"><<a href="mailto:hverbeet@gmail.com" target="_blank">hverbeet@gmail.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 24 June 2013 16:44, Matteo Bruni <<a href="mailto:matteo.mystral@gmail.com">matteo.mystral@gmail.com</a>> wrote:<br>
> @@ -1673,6 +1673,8 @@ struct texture_stage_op<br>
>      unsigned                aarg2 : 8;<br>
>      unsigned                aarg0 : 8;<br>
><br>
> +    DWORD                   constant;<br>
> +<br>
>      struct color_fixup_desc color_fixup;<br>
>      unsigned                tex_type : 3;<br>
>      unsigned                dst : 1;<br>
><br>
> You don't need this if you use uniforms.<br>
> Also adding a test would be nice probably.<br>
><br>
</div>Yeah, that's the main issue. We don't want a shader for every possible<br>
set of per-stage constant values. The one you missed is that if we<br>
implement this, we should also set the appropriate caps bit.<br></blockquote><div><br></div><div>Spin Tires does not seem to check this caps. That's why I missed it first.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
The most obvious issue is of course the subject line. Sure, typos<br>
happen, but it's just plain sloppy. </blockquote><div><br></div><div>It's a typo yes. You can easy deduced this with the patch description. <br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
At least try to pretend you put<br>
some effort into making sure the patch is as good as it can be before<br>
submitting it.<br></blockquote><div> </div></div>I could have send the patch to wine-devel first but would I get something longer that "several issues" as feedback?<br></div><div class="gmail_extra">I can certainly improve my patches and you can certainly improve your way of giving feedback but that would be great<br>
</div><div class="gmail_extra">if you could avoid this kind of commentary. This is not usefull. At least for me. For technical stuff you can be as verbose as you want. I'm full open.<br></div></div>