<div dir="auto"><div>Thanks for clarification. I meant vk_ icd specific calls. Was early in the morning. The wine_ calls obviously need to be internal. Okay so if private means windows dolls can still call then that's fine then. <br><br><div class="gmail_quote"><div dir="ltr">On Fri, Jul 27, 2018, 7:46 AM Jacek Caban <<a href="mailto:jacek@codeweavers.com">jacek@codeweavers.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 07/27/2018 04:38 PM, Roderick Colenbrander wrote:<br>
> Hi Jacek,<br>
><br>
> Why are you marking the ICD functions as private? This would break<br>
> native vulkan-1 support, which requires these calls.<br>
<br>
I'm adding a reason for winevulkan to be linked by winelib applications<br>
in patch 1/2. Such applications shouldn't be linking to any of ICD nor<br>
wine_* functions.<br>
<br>
Note that -private means only that they will not be exposed by<br>
importlib. Those functions will still be exported by name by DLL and<br>
vulkan-1 may still forward to them. Do you expect that we will want to<br>
call them by name from C code outside winevulkan in the future? If yes,<br>
then we can drop this patch.<br>
<br>
Thanks,<br>
Jacek<br>
</blockquote></div></div></div>