Tunable cost names containing string ", "

From formats.h:

/*
 * Descriptions (names) of tunable cost parameters for this format
 *
 * These names shouldn't contain ',', because ", " is used
 * as a separator when listing tunable cost parameters
 * in --list=format-details and --list=format-all-details.
 * The sequence of names should match the sequence of functions
 * returning tunable cost values.
 */
	char *tunable_cost_name[FMT_TUNABLE_COSTS];

Nevertheless, several formats have tunable cost names which contain , (especially formats which report a list of different algorithms as a tunable cost.

I intend to fix the offending formats, e.g. by replacing

"algorithm [0=AES, 1=TwoFish, 2=ChaCha]"

with

"algorithm [0=AES 1=TwoFish 2=ChaCha]"

and then enhancing format self test (fail if a tunable cost contains , (or even just ,), and fail if a tunable cost name i empty or if the number of elements in tunable_cost_value[] and tunable_cost_name[] differ.

Any objections?

Should this test be executed only for `full_lvl >= 0
Probably it is best to but all these tunable cost specific into a separate function.

Anything else related to tunable costs that needs to be tested?

Start with the tunable_cost_name definition and comments in formats.h, then locate the format self test and the --list=format-details output it protects. Identify format names containing ", ", update the offending entries, and make the self test reject comma-containing or empty names and mismatched tunable cost arrays.