<div dir="ltr"><div class="gmail_quote">On Fri, Oct 19, 2012 at 5:25 PM, Petr Mladek <span dir="ltr"><<a href="mailto:pmladek@suse.cz" target="_blank">pmladek@suse.cz</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Fri, 2012-10-19 at 15:27 +0200, Lior Kaplan wrote:<br>
> (replying to the list, as I don't think gerrit is the right platform<br>
> for discussions about patches)<br>
<br>
</div>Yup, it is a bit strange. I think that you could press the "Review"<br>
button and just leave the comment. Well, you can't do inline comments<br>
this way.<br></blockquote><div><br>Indeed (:<br> <br></div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="im">
> Hi Petr,<br>
><br>
> I've done the patch in <a href="https://gerrit.libreoffice.org/883" target="_blank">https://gerrit.libreoffice.org/883</a> to be as<br>
> minimal as possible.<br>
><br>
> I guess that it's indeed an unneeded repetetion, but as it appear<br>
> about 100 times in the same file (and effect different languages), I<br>
> would prefer to leave removing the condensed form to another change.<br>
<br>
</div>I see the following:<br>
<br>
$> grep lucidasansunicode officecfg/registry/data/org/openoffice/VCL.xcu<br>
| wc -l<br>
3<br>
$> grep "Lucida Sans Unicode"<br>
officecfg/registry/data/org/openoffice/VCL.xcu | wc -l<br>
68<br>
<br>
=> "Lucida Sans Unicode" is used many times and "lucidasansunicode" only<br>
3 times.<br></blockquote><div><br>But "lucidasans" is used 100 times. <br> </div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
You could see that "lucidasansunicode" is used only in the second part<br>
of the VCL.xcu file where you define substitutes for particular fonts.<br>
Only this second part uses the lowercase-condensed font names.<br></blockquote><div><br>And I don't want to lose this part due to the condensed/regular form change you've suggested without more thorough check.<br>
</div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
=> I still think that you should use only the normal font name (Lucida<br>
Sans Unicode) in the first part where you define fonts for each locale.<br>
<div class="im"><br>
<br>
> If someone has any other clean up for<br>
> officecfg/registry/data/org/openoffice/VCL.xcu let me know, and I'll<br>
> be happy to do them.<br>
<br>
</div>I am not aware of any planed clean up. Well, it would be great to rework<br>
the code and use the same font names everywhere and avoid the confusion.<br></blockquote><div><br>As you can see, there's place to unify the font's writing, but I would prefer to leave it to another (larger) patch.<br>
</div></div><br>Thanks for the comment...<br><br>Kaplan<br></div>