In fact there is another problem just 2 lines below:<br><b>if (b_retain) break;</b><br>It used to be inside <b>switch{} </b>block, so break was intended to get out of switch. During <a href="http://trac.videolan.org/vlc/changeset/23654">
http://trac.videolan.org/vlc/changeset/23654</a> switch was eliminated, so now break interrupt <b>for{}</b>. For the user it looks like when VLC is when you are saving preferences from the corresponding window - it works fine, but when you are close VLC and it does automatic save, it stops the process after saving first string setting for each section. Attached is cumulative patch that fixes both problems.
<br><br><div class="gmail_quote">On Jan 9, 2008 5:28 PM, Andrew Zaikin <<a href="mailto:andrew.zaikin@gmail.com">andrew.zaikin@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
4th parameter of <b>config_Write()</b> called <b>comment</b> and determine whenever setting will be prefixed by "#" in config file<br>All occurrence of <b>config_Write() </b>except patched treat this parameter as "is value remains the same", but for strings we are passing value of 
<b>modified</b> variable which is logical inversion.<br>So it looks like we were writing string values, that was non-modified and commenting modified<br>
</blockquote></div><br>