[http://sourceforge.net/p/ufoai/feature-requests/368 Item 368] imported from sourceforge.net tracker on 2013-01-28 20:18:20

it would be nice to have parameter negation for -check/fix subparameters. for example :
ufo2map -check -con
would skip 'contained' checks.

reasoning - sometimes a particular check/fix might be broken or suboptimal (for example, current 'con' lists brushes inside actorclips, thus polluting the output alot...) or might not be desired for some other reason.
===== Comments Ported from Sourceforge =====

====== blondandy (2008-09-01 07:39:51) ======

Logged In: YES
user_id=1760416
Originator: NO

i cannot think of a reasonable way of implementing this (that would not take longer than just fixing the proble).

i suggest that if there is a particular test that is causing a problem, it is removed from "all" temporarily, like ndr still is. then it would have to be specifically included on the command line.

(i think ndr is working for faces covered by single faces, btw.)

====== richlv (2008-09-01 07:55:03) ======

Logged In: YES
user_id=891377
Originator: YES

is that because prefixing subparams with '-' would make them look like params ?
what about using different notation, like "nobru" or something ?
====== blondandy (2008-09-01 08:19:16) ======

Logged In: YES
user_id=1760416
Originator: NO

in the structure that stores items such as whether we are in fix mode, whether we are checking nodraws etc. if all is set then these are overidden. i would have to change these from qbooleans (which are qtrue or qfalse) to having 3 states (check, not_check, and not_check_even_if_all).

The function in ufo2map.c that parses args is long and ungly. doubling the length of the subparameter part of it would make it uglier and harder to maintain.

I do not think it is worth the effort for a feature that would only be useful in a transitional period.

It is easy to exclude a subparameter from all. look at the @todo in ufo2map.c. If a subparameter is really pissing you off you could comment out the "|| config.chkAll" part of the if() expression. (Trying to find the exact line for you, but the web interface to the repo seems to be down.)