<p>After a discussiong regarding this patch (with j-b in <code>#videolan</code> @ <code>freenode</code>) it has come to my attention that the identifiers that I (mindlessly) wanted to "deprecate" are coming directly from native windows headers, and as such deprecation would be <strong>very</strong> hard.</p>
<p>In other words; this patch should be thrown in the trash.</p>
<pre><code>  ____.-.____
 [___________]
(d|||||||||||b)
 `|||TRASH|||`
  |||||||||||
  |||||||||||
  |||||||||||
  |||||||||||
  `"""""""""`</code></pre>
<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">
<ul>
<li><p>renamed include-guard since the old one was a reserved identifier</p></li>
<li><p>replaced comment at <code>#endif</code> corresponding to include-guard</p></li>
<li><p>Added preprocessor directives to issue a warning upon usage of identifiers that are reserved according to the ISO C Standard.</p>
<p>These checks should be removed when we are confident that they are still not used in the codebase. From this on forward the following directives should be used (if ever);</p>
<ul>
<li><code>VLC__SKIP_WAVEFORMATEX</code> instead of <code>_WAVEFORMATEX_</code></li>
<li><code>VLC__SKIP_WAVEFORMATEXTENSIBLE</code> instead of <code>_WAVEFORMATEXTENSIBLE_</code></li>
<li><code>VLC__SKIP_WAVEHEADER</code> instead of <code>_WAVEHEADER_</code></li>
<li><code>VLC__SKIP_BITMAPINFOHEADER</code> instead of <code>_VLC_BITMAPINFOHEADER_</code></li>
<li><code>VLC__SKIP_RECT32</code> instead of <code>_RECT32_</code></li>
<li><code>VLC__SKIP_REFERENCE_TIME</code> instead of <code>_REFERENCE_TIME_</code></li>
<li><code>VLC__SKIP_VIDEOINFOHEADER</code> instead of <code>_VIDEOINFOHEADER_</code></li>
<li><code>VLC__SKIP_RGBQUAD</code> instead of <code>_RGBQUAD_</code></li>
<li><code>VLC__SKIP_TRUECOLORINFO</code> instead of <code>_TRUECOLORINFO_</code></li>
<li><code>VLC__SKIP_VIDEOINFO</code> instead of <code>_VIDEOINFO_</code></li>
</ul></li>
</ul>
</blockquote>