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