On Tue, Oct 29, 2024 at 08:22:29PM +0100, Jean-Marc Lasgouttes wrote: > I am not sure what you mean, Pavel. Not using default: is better in general. > We should use it only in cases where the number of enum members is too > large(for example in dispatch+)).
I thought that we generally do default, but you are right, looking at the code, it's specific to dispatch routines. Apparently I am too paranoid people for doing dirty stuff like directly passing ints... Is using at least do return "" at the very end of the function or use some local variable like in toHTMLSpanClass() so that we always return something well defined considered sensible here? Pavel -- lyx-devel mailing list lyx-devel@lists.lyx.org https://lists.lyx.org/mailman/listinfo/lyx-devel