<p>These patches can be ignored until we have decided upon a suitable naming convention for include-guards.</p>
<p>There is currently an ongoing discussion of the matter in <code>#videolan</code>, and I will try to keep this thread updated with the consensus reached.</p>
<p>Jean-Baptiste Kempf proposed the following convention (uppercase):</p>
<ul>
<li><code>VLC_${module-name}_${filename-without-ext}_${extension}</code></li>
</ul>
<p>I personally think that it would be a good idea to also include a trailing underscore to signal that such identifier is never to be used outside the file it was introduced in.</p>
<p>Stay tuned for updates.</p>
<hr />
<p>On 16/02/20 23:20, Filip Roséen wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<p>These patches all boil down to fixing issues where reserved identifiers are used, making the codebase ill-formed according to the C (WG14) ISO Standard.</p>
<ul>
<li>http://www.open-std.org/jtc1/sc22/wg14/www/docs/n1124.pdf (section: 7.1.3/1)</li>
<li>http://www.gnu.org/software/libc/manual/html_node/Reserved-Names.html</li>
<li>http://c-faq.com/decl/namespace.html</li>
</ul>
<p>In short, removal/fix of the following type of identifiers:</p>
<ul>
<li>identifiers in the global scope that start with an underscore</li>
<li>identifiers (everywhere) that start with an underscore followed by a uppercase letter</li>
<li>identifiers (everywhere) that start with two underscores</li>
</ul>
</blockquote>