Georg Baum wrote:
Am Mittwoch, 13. September 2006 17:41 schrieb Abdelrazak Younes:
Angus Leeming wrote:
Note that keeping std::string for the format name will enable you to
enforce
the distinction at compile time. Even clearer if you follow Georg's
suggestion
of
    typedef std::string lyx::asciistring;

I don't think it was from me, but that is not so important :-)

Actually I think it was me but I did not choose this name at the time :-)


Why not... I would even go further:

      typedef std::string lyx::keystring;

I would even go further:

class Format {
public:
        typedef std::string name_type;
};

The fact that it is a string is not important IMO, so it should not be in the typename. Logically I think it belongs to the Format class. All functions that only deal with format names then take a Format::name_type argument, and because it is std::string we can still use general string utility functions with it.

Yep, makes sense.

Abdel.

Reply via email to