On Fri, Apr 14, 2006 at 09:57:21AM +0200, Georg Baum wrote:

> Am Mittwoch, 12. April 2006 23:39 schrieb Enrico Forestieri:
> > The output of the command is like this:
> > 
> > $ mount --show-cygdrive-prefix
> > Prefix              Type         Flags
> > /cygdrive           system       binmode
> > 
> > so, the inner split returns "/cygdrive           system       binmode"
> > whereas the outer split assigns "/cygdrive" to cygdrive and returns
> > "          system       binmode" which I simply discard, and the (void)
> > makes that clear. I think it is a very compact and effective code ;-)
> 
> It may be compact, but that does not always mean good. The problem is that 
> one does not understand what it does from reading it. This is not only 
> your fault, the strange interface of split() contributes, too.
> 
> > > The rest looks good, and if you give me a log message and fix the 
> above I 
> > > will put it in.
> > 
> > I think you don't love APL ;-)
> 
> I never saw it in a real project, so I don't know whether I would love 
> it :-)

I had a collegue which used and loved that programming language.
I was really impressed by its compactness but was much more impressed
by its crypticity ;-)

> > Is it sufficient adding a comment? Changing that would require more
> > than one line for no good...
> 
> If you add a comment with the mount output then it would be OK with me.

OK, then I will do that.

-- 
Enrico

Reply via email to