
<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">

<br><div><div>On Dec 2, 2007, at 5:25 AM, Jean-Baptiste Kempf wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><p style="margin: 0.0px 0.0px 0.0px 0.0px"><font face="Helvetica" size="3" style="font: 12.0px Helvetica">OK. So what should we do ?</font></p> </blockquote></div><br><div>Well, we could try to open a ticket for it. BTW, most of the enum we already have in VLC follow the #define naming scheme, which is... bothering.</div><div><br class="webkit-block-placeholder"></div><div>"Rationalize preprocessor usage"</div><div><br class="webkit-block-placeholder"></div><div> - Use enum where possible instead of #define</div><div>(We have to be careful to understand that sometimes #ifdef the const is useful.)</div><div><br class="webkit-block-placeholder"></div><div><div>- Avoid #define function where possible, and prefer static inline</div></div><div><br class="webkit-block-placeholder"></div><div>We could add:</div><div>- Avoid certain static inline, if the performance impact isn't noticable.</div><div><br></div><div>Pierre.</div><div><br class="webkit-block-placeholder"></div><div><br class="webkit-block-placeholder"></div></body></html>