Hi Stepan,

* Stepan Kasal wrote on Tue, Aug 16, 2005 at 08:02:31PM CEST:
> one minor nit:
> 
> On Tue, Aug 16, 2005 at 07:36:07PM +0200, Ralf Wildenhues wrote:
> > +  at_groups=`echo "$at_groups$as_nl$at_groups_all" |
> > +    awk ['BEGIN { FS = ";" }
> > +   !/ / { selected[$ 1]  = 1 }
> 
> since the field splitting is effectively switched off, $0 might
> be more readable here.  Like this:

Sure.  Feel free to adjust the patch accordingly.

>     ...
>     awk ['BEGIN { FS = ";" # Effectively switch off field splitting.
>        }
>       !/ / { selected[$ 0]  = 1 }
>     ...

Cheers,
Ralf


Reply via email to