<p>I just tested to generate documentation through doxygen after supplying the original patch, and everything works as it shall:</p>
<ul>
<li>http://atch.se/vlc-devel_doxygen_result.png</li>
</ul>
<p>Given the fact that the original patch is far more clean than the alternative attached earlier in this discussion, my vote goes to the original.</p>
<ul>
<li>https://mailman.videolan.org/pipermail/vlc-devel/2016-February/106215.html</li>
</ul>
<hr style="height:1px;margin-bottom:20px;background-color:#ddd;color:#ddd" />
<p>On 16/02/22 20:28, Filip Roséen wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<p>I know that. You are not answering my question.</p>
</blockquote>
<p>It depends on the doxygen configuration, but since it is used elsewhere in <code>include/vlc_plugin.h</code> I did not think that it would be an issue. An alternative patch that simply wraps the <code>typedef</code> in a conditional <code>extern "C"</code> is attached to this email.</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<p>Also please avoid HTML formatting.</p>
</blockquote>
<p>I send my emails (not this one) as both text/plain and text/html, as can be seen if you view my messages at the <code>vlc-devel</code> archive (notice that it states that there is an <em>alternative</em> <code>text/html</code> attachment).</p>
<ul>
<li>https://mailman.videolan.org/pipermail/vlc-devel/2016-February/106215.html</li>
<li>https://mailman.videolan.org/pipermail/vlc-devel/2016-February/106217.html</li>
</ul>
<p>It is up to your email-client which of the ones it displays when you read the message (as an example I have mine defaulting to text/plain).</p>
<p>I was not aware of guidelines stating that one should not use <code>multipart/alternative</code> on this mailing-list.</p>
<hr style="height:1px;margin-bottom:20px;background-color:#ddd;color:#ddd" />
<p>On 16/02/22 21:11, Rémi Denis-Courmont wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<p>I know that. You are not answering my question.</p>
<p>Also please avoid HTML formatting.</p>
</blockquote>
</blockquote>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<p>diff –git a/include/vlc_plugin.h b/include/vlc_plugin.h index 46abe4e..e4378c2 100644 — a/include/vlc_plugin.h +++ b/include/vlc_plugin.h @@ -234,8 +234,17 @@ enum vlc_module_properties # define EXTERN_SYMBOL #endif</p>
<p>+#if defined (__cplusplus) +extern “C” { +#endif + typedef int (<em>vlc_set_cb) (void </em>, void *, int, …);</p>
<p>+#if defined(__cplusplus) +} +#endif + + #define vlc_plugin_set(…) vlc_set (opaque, NULL, <strong>VA_ARGS</strong>) #define vlc_module_set(…) vlc_set (opaque, module, <strong>VA_ARGS</strong>) #define vlc_config_set(…) vlc_set (opaque, config, <strong>VA_ARGS</strong>)</p>
</blockquote>