<div dir="ltr"><div dir="ltr">On Sun, Apr 24, 2022 at 5:26 AM Alexandre Julliard <<a href="mailto:julliard@winehq.org">julliard@winehq.org</a>> wrote:<br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>Austin English <<a href="mailto:austinenglish@gmail.com" target="_blank">austinenglish@gmail.com</a>> writes:<br>
<br>
>  At the same time, I agree of course that just the contributor's desire <br>
>  to work on something is enough to consider submitted work. For this <br>
>  patch what kind of feedback would you expect?<br>
><br>
> I think the feedback/review, in general, has been fair/valid. I am<br>
> disagreeing in how explicit Wine is being, as a project, for testing<br>
> patches before submitting upstream.<br>
<br>
The issue is not testing the patch, it's demonstrating that there's a<br>
need for it in the first place.<br></div></blockquote><div><br></div><div><div> Can we clarify what is needed to demonstrate that this patch is needed?<br><br></div><div>I.e., it would  be helpful, particularly to new/infrequent developers, to be able to note when it's necessary for a developer to explicitly test a patch. I.e., if it involves IE internals rather than win32, if the API is not on MSDN, etc.<br></div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>
That has been the policy forever, we only implement things that are<br>
actually used by a real application. There are tons of things in the<br>
Windows API that are used only internally, or no longer relevant, or<br>
simply never used at all. Adding them only increases our workload for<br>
no benefit.<br></div></blockquote><div> </div><div>Yes, I'm aware, having been bit by it a few times myself, but thanks for the reminder ;).<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div>-- <br>
Alexandre Julliard<br>
<a href="mailto:julliard@winehq.org" target="_blank">julliard@winehq.org</a><br>
</div></blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr">-Austin<br>GPG: 267B CC1F 053F 0749 (expires 2024/02/17)</div></div></div>