>> - // return the widest symbol
>> + // return the widest symbol as w times a "W"
>> + for (int n = 1; n <= w; ++n)
>> + symb = symb + "W";
>
> I do not understand this, and I think this is wrong. What if "iiii" is indeed
> my longest label?
"iiii" is _always_ the shortest 4 character string, while "WWWW" is always the
widest one.
> + // assure that the width is never below the predefined
> Why do you forbid to go under the predefined?
To have a consistent output. In most cases the 1 cm width is enough, so we have to assure that this
width is not undergone, for example when you have several nomenclatures. I'll send you this evening
a LyX example file, to show what I mean.
> Also, I do not like the complicated ifthen statement at all.
Why is ifthen complicated?
> I've changed this to use theFontMetrics instead. I think this is a much better
> approximation (and the best we can get).
I don't think so, because you don't know the font metrics of the output font, and this is what
counts. My solution assured, the there is always enough space, no matter what font is used. The only
case where this might produce a too large width is when your widest symbol is "iiiiiii", but this is
an extreme and absurd example.
>> > + // assure that the width is never below the predefined
>>
>> Why do you forbid to go under the predefined? Also, I do not like the
>> complicated ifthen statement at all.
>
> Now that we have a param "auto" and "none", I think this should go. If the
> user selects "auto", we should always use that and not care about the
> predefined length.
No we have to assure that the width is never below the predefined value. But to be correct, I'll
change the hardcoded "1cm" to the one that is actually defined in nomencl.cfg, that's only one more
line.
regards Uwe
