<p>Hi Hugo,</p>
<p>On 16/08/02 19:05, Hugo Beauzée-Luyssen wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code>Sorry for the lack of comment... basically the idea is this change allows
one to see the log on some platform that would swallow stdout/stderr,
making debugging easier. It seems weird that this wasn't done before, yet
I can't see a reason for it, hence the patch.</code></pre>
</blockquote>
<p>Given that <code>config_LoadCmdLine</code> is invoked prior to <code>vlc_LogInit</code>, using <code>msg_*</code> for messages related to processing the command-line options would currently lead to such messages not being displayed, through <code>vlc_vaLogDiscard</code>, if no <em>logger-module</em> is found usable.</p>
<p><code>vlc_LogPreInit</code> enables log messages to be stored in memory until a logger-module is available, meaning that diagnostics will not be displayed until after <code>vlc_LogInit</code> is called (if ever).</p>
<p>In other words; if an invalid –logger option is supplied, or if loading of the logger-module fails for some other reason, the diagnostic related to parsing command-line options would be lost.</p>