While cppcheck did not turn out any suspicious error messages, using
the "performance" flag highlighted several nitpicks in three categories
* do not use it++ for iterators, ++it is better
* do not use size() to test for emptyness, empty() is here
* do not use "const T" as a function parameter, "const & T" is better
I doubt that any of these is a real performance problem, but the code is cleaner anyway.
- get rid of function lengthAutoToWidgets, which was only used by GuiGraphics
- new variant of lengthToWidgets that takes a docstring
* Validator.{cpp,h}:
- the auto text in the Auto validators are customizable
* GuiGraphics.cpp:
- make the "auto" string translatable
git-svn-id: svn://svn.lyx.org/lyx/lyx-devel/trunk@28165 a592a061-630c-0410-9148-cb99ea01b6c8
