Uwe Stöhr wrote: > I have there used a hack where I code the case of no option to the length > "-999col%". This works but I am not happy with this. There must be a way to
It is indeed very ugly. In case you can't find better solution at least define some constant sensible_name=-999col% and use this constant, do not spray such silly strings all over the place. > + widthCB->setEnabled(itype == "makebox" > + || (outer == "Boxed" && itype == "none")); I see this type of hardcoded structure used multiple times. Is it some semantic structure which could be generally rewritten by bool some_box_feature(itype,outer)? In case this is file format change, some things are still missing for review. Pavel BTW you seem to miss multiple messages on list. Do you read it or react only when directly CC-ed?
