[coreboot] [PATCH] v3: print name of compression algorithm

[Peter Stuge]

On Sun, Feb 17, 2008 at 12:34:24AM +0100, Carl-Daniel Hailfinger wrote:
> +enum compalgo {
> +	none = 0,
> +	lzma = 1,
> +	nrv2b = 2,
> +	zeroes = 3,
> +	/* invalid should always be the last entry. */
> +	invalid
> +};

It's fairly common to uppercase these. Do we want that too?


> +	const char *algoname = algo_name[(archive->compression >= invalid)
> +					? invalid : archive->compression];

Not sure about invalid. I would be less worried if invalid was set to
some explicit value, e.g. -1. The current proposal means we can never
remove algorithms.


//Peter