Le 14/12/2015 12:05, Guillaume Munch a écrit :
> You're right, see now second patch attached. It's a fairly simple
> change, only adding some information to Georg's displayColAlign(), in
> addition to some mechanical refactoring and disabling of horizontal
> alignment buttons which were wrongly enabled. See the new commit log.

Thanks for the patches. A couple remarks:
- in switches it would be better to avoid using default:. This way, when
adding a new hull type, we'll get a warning for each un-handled case and
we'll have to think about the right value.
- if defaultColAlign and defaultColSpace are broken in some way, the
least we shell do is add a FIXME in the source. Deciding what to do with
them would be even better.

Unfortunately I do not have time until end of week for testing.

JMarc

Reply via email to