[Libreoffice] Easy hack: expunge duplicate enumerations in vcl

Caolán McNamara caolanm at redhat.com
Tue Apr 5 02:57:18 PDT 2011


On Mon, 2011-04-04 at 23:18 +0200, Christina Roßmanith wrote:
> As far as I understand the description, psp::width::UC shall be
> removed. But shall the enum from sft.hxx or from vclenum.hxx be kept?
> 60:40 for vclenum.hxx I would guess.

Yeah, the psprint stuff used to live somewhere else, so the enum was
added there and the mapping stuff to allow that build to work, psprint
got folded into vcl, so no need for the dup of that one anymore. So 

a) the psp:: ones definitely go, and
b) definitely get replaced with the vclenum.hxx ones.

The sft.hxx stuff on the other hand are from another piece of code that
got merged into vcl from some third party source IIRC, and describe the
values used in the truetype font format, I think those values match the
physical numbers stored in those files so they should, for now anyway,
get left alone, and the mappings from/to them and the vclenum values
left alone.

> why has tools its own WIDTH_ULTRA_CONDENSED?

hum, wasn't aware of that. The original commit there says "move this
from vcl to get xmloff free of vcl stuff", so (as a follow up maybe) it
would seem to make sense to e.g. remove the dups from vclenum.hxx and
have it include tools/fontenum.hxx to provide those.

>  And if vcl is an acronym: what does it stand for?

I think its Visual Components Library.

C.



More information about the LibreOffice mailing list