Unfortunately, the "spaces" are not removed when rendered.  Here are URLs of screenshot images of the equalizer panel that 
demonstrate the rationale behind my fixes:<br><br>Before (as it is right now in trunk):  <a href="http://tinypic.com/r/2u7nn7b/6">http://tinypic.com/r/2u7nn7b/6</a><br>After (with posted patch):  <a href="http://tinypic.com/r/2qa7rfd/6">http://tinypic.com/r/2qa7rfd/6</a><br>
After (with just the alignment issue fixed):  <a href="http://tinypic.com/r/ix87xs/6">http://tinypic.com/r/ix87xs/6</a><br><div class="yj6qo ajU"><div id=":144" class="ajR" tabindex="0"><img class="ajT" src="https://mail.google.com/mail/u/0/images/cleardot.gif"></div>
</div><br><br><div class="gmail_quote">On Sun, Mar 10, 2013 at 11:58 AM, Francois Cartegnie <span dir="ltr"><<a href="mailto:fcvlcdev@free.fr" target="_blank">fcvlcdev@free.fr</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Le 10/03/2013 17:54, Ron Wright a écrit :<br>
<div class="im">> What I meant to say was that the bands in the actual equalizer module do<br>
> not need to be changed, but in the GUI, the strings corresponding to the<br>
> two lowest ISO frequencies are simply too long and need to be shortened in<br>
> some fashion.  Otherwise, I could submit an alternative patch that fixes<br>
> only the alignment issue (caused by the extra spaces in the 1 kHz and 2 kHz<br>
> labels) because this issue is much more important.<br>
<br>
</div>We don't need "spaces" or to change text at all.<br>
<br>
foreach( slider )<br>
        computemaxwidth();<br>
foreach( slider )<br>
        setMinimumSize( maxwidth, h );<br>
<div class="HOEnZb"><div class="h5"><br>
Francois<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</div></div></blockquote></div><br>