Skip to contentSkip to conversation

minor lvchange option validation refactor

PATCH New Petr Rockai 2 participants 2 messages Jun 24, 2010

lore
Share & tools
lore
#0 lore raw

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.

#1 lore raw

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.