<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<p>Hi Alex,<br>
</p>
<br>
<div class="moz-cite-prefix">On 12/7/17 8:34 PM, Alex Henrie wrote:<br>
</div>
<blockquote type="cite"
cite="mid:CAMMLpeQ_1ZfhbNJ2ay-NT-4pJ6TnrBCyANpqdmF0+p7USHT4YA@mail.gmail.com">
<pre wrap="">2017-12-07 12:20 GMT-07:00 Jacek Caban <a class="moz-txt-link-rfc2396E" href="mailto:jacek@codeweavers.com"><jacek@codeweavers.com></a>:
</pre>
<blockquote type="cite">
<pre wrap="">On 06.12.2017 20:13, Alex Henrie wrote:
</pre>
<blockquote type="cite">
<pre wrap="">+
+#pragma makedep register
+#pragma makedep regtypelib
+
+#include "dhtmled.idl"
</pre>
</blockquote>
<pre wrap="">
This won't work without at least threading() specified in dhtmled.idl.
</pre>
</blockquote>
<pre wrap="">You're right; I noticed that before but did not notice this time
because I was testing incorrectly. I'm happy to resend patch 1 with
threading attributes.</pre>
</blockquote>
<br>
It would be fine as a separated patch as well. In fact, given that
we're entering code freeze today and v4 version still doesn't get
registration right, I would prefer to take v3 patch now and defer
the rest until after code freeze.<br>
<br>
<blockquote type="cite"
cite="mid:CAMMLpeQ_1ZfhbNJ2ay-NT-4pJ6TnrBCyANpqdmF0+p7USHT4YA@mail.gmail.com">
<blockquote type="cite">
<pre wrap="">It should probably also use progid and vi_progid.
</pre>
</blockquote>
<pre wrap="">There aren't progid or vi_progid attributes in the Microsoft IDL.</pre>
</blockquote>
<br>
Yeah, they are widl extensions for registration.<br>
<br>
<blockquote type="cite"
cite="mid:CAMMLpeQ_1ZfhbNJ2ay-NT-4pJ6TnrBCyANpqdmF0+p7USHT4YA@mail.gmail.com">
<pre wrap="">If we add them, what values should they have?
</pre>
</blockquote>
<br>
See relevant registry values on Windows (or maybe native installed
in Wine).<br>
<br>
Thanks,<br>
Jacek<br>
</body>
</html>