On Sun, Aug 12, 2007 at 08:21:44PM +0200, Bastian Blank wrote: > On Sun, Aug 12, 2007 at 06:27:17PM +0200, Guido Guenther wrote: > > + [state] = "DM_STATE=\"%s%s\"\n", > > + [tablestate] = "DM_TABLE_STATE=\"%s%s%s\"\n", > > Why use freetext fields if you can set it in a splitted way? > > DM_STATE for example shows both suspended and read-only state without > making in properly parsable. This is just does what the patch I based this on does, I'd rather split this into separate fields too. Cheers, -- Guido -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel