[PATCH] minor lvchange option validation refactor

[PATCH] minor lvchange option validation refactor

[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).

-------------- 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] minor lvchange option validation refactor

[Dave Wysochanski]

On Thu, 2010-06-24 at 12:57 +0200, Petr Rockai wrote:
> 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).
> 
> 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.