<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">Hi Emil,</div><div class="gmail_quote"><br></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>
</span>Please include your follow-up reply/context as commit message.<br></blockquote><div><br></div><div>Will do.</div><div><br> </div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span>> +// cleanup for failed screen creation<br>
> +PUBLIC void swr_destroy_screen_internal(st<wbr>ruct swr_screen **screen);<br>
<br>
</span>I'm fairly sure you don't need to make this function public. It's used<br>
within the same binary - <a href="http://libmesaswr.la" rel="noreferrer" target="_blank">libmesaswr.la</a>. Regardless if built-in or<br>
separate DSO's are used.<br></blockquote><div><br></div><div>You are, of course, correct.  I was conflating static with not public and thinking that since it's not in the same translation unit then it can't be static so it must be public, which is certainly not the case.</div><div> </div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
With the above two suggestions:<br>
<br>
Cc: <a href="mailto:mesa-stable@lists.freedesktop.org" target="_blank">mesa-stable@lists.freedesktop.<wbr>org</a></blockquote><div><br></div><div>I purposely left this out of mesa-stable since it requires the previous two commits that initially refactored swr_create_screen to allow for builtin arch, which were not part of stable.</div><div><br></div><div><font color="#888888">- Chuck</font><br></div></div><br></div></div>