On Wed, Dec 21, 2011 at 10:49:15AM -0000, TJ wrote:
> Your patch looks good - although you could drop the use of $found to
> track detection and use "break 2" which would avoid those extra 4 lines
> of code.
Not at all; 'break 2' has the effect that no other lines from /etc/crypttab
will be processed, which is definitely not what we want.
--
Steve Langasek Give me a lever long enough and a Free OS
Debian Developer to set it on, and I can move the world.
Ubuntu Developer http://www.debian.org/
<email address hidden> <email address hidden>
On Wed, Dec 21, 2011 at 10:49:15AM -0000, TJ wrote:
> Your patch looks good - although you could drop the use of $found to
> track detection and use "break 2" which would avoid those extra 4 lines
> of code.
Not at all; 'break 2' has the effect that no other lines from /etc/crypttab
will be processed, which is definitely not what we want.
-- www.debian. org/
Steve Langasek Give me a lever long enough and a Free OS
Debian Developer to set it on, and I can move the world.
Ubuntu Developer http://
<email address hidden> <email address hidden>