minor lvchange option validation refactor
PATCH 1/1 minor lvchange option validation refactor
Hi,
I would normally check this in right away, but on a second thought (since I got the change wrong the first time), an extra reading of the patch can't hurt (just check with good old De Morgan's). It is passing testsuite (now).
-------------- next part --------------
A non-text attachment was scrubbed...
Name: lvchange-opts.diff
Type: text/x-diff
Size: 2647 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/lvm-devel/attachments/20100624/54ba7446/attachment.bin>
-------------- next part --------------
Thanks!
Petr.
PS: I ran into this sorting through stale mail. I may have a few more such outlandish ideas as I sort through more of it.
PATCH 1/1 minor lvchange option validation refactor
Quoted 7 lines · Petr Rockai
Hi,
I would normally check this in right away, but on a second thought (since I got the change wrong the first time), an extra reading of the patch can't hurt (just check with good old De Morgan's). It is passing testsuite (now).
Quoted 6 lines
Thanks!
Petr.PS: I ran into this sorting through stale mail. I may have a few more such outlandish ideas as I sort through more of it.
--
Ack. Read it through, looks like a good refactor that clarifies the code.