Re: Various issues

On Wed, 13 Aug 2003, Bram Moolenaar wrote:
> This is what you would use now:
>
> 	s = $-source
> 	@if s == "foo.c":

Yes, that's what I use now. And it works fine; I wouldn't want to add
anything in Python to do that. It should be documented though .. I'll add
it to my list of stuff to find a spot for.


> I think it just makes things more complicated.  My intention was to only
> use Python for flow control and where it's really needed.  Duplicating
> things that you can also do with Aap commands is to be avoided.  Only
> when there is a good reason it would be implemented as a Python
> function.

Agreed.

> > I've coded up a replacement of sorts, that has the side effect that in the
> > sufreplaced list all the attributes are gone. It's on another machine
> > though, I'll send it in on the weekend.
>
> Hmm, I don't think the side effect of dropping the attributes is
> desired.  It's always better to have a function do one thing.  It should
> be fairly simple to keep the attributes, using str2dictlist() and
> dictlist2str().

O yeah. Then I can examine key "name" and modify that. OK.

> > It's something of an automake holdover. :child $SUBDIRS seems much more
> > convenient that "for i in var2list(SUBDIRS): :child $i". Maybe it never
> > happens.
>
> Ah, if you assign the list of subdirectories to a variable I can see the
> point of ":child" accepting a list of recipes.  But you might as well
> call it $CHILDREN and give the recipe names.  Note that they could be in
> the same directory as the current recipe.  Mixing file names with
> directory names may be a bit confusing.

Agreed. In the SUBDIRS case, AAP's existing rc-expansion can handle it
with :child $*SUBDIRS/main.aap (um, can it? does that work?). I'd suggest
that adding support for multiple children in one command is useful, but
the implicit main.aap is not.

-- 
 Adriaan de Groot    adridg-FlD2LfDziEhmR6Xm/[email protected]     Kamer A6020     024-3652272
GPG Key Fingerprint 934E 31AA 80A7 723F 54F9  50ED 76AC EE01 FEA2 A3FE
               http://www.cs.kun.nl/~adridg/research/



-------------------------------------------------------
This SF.Net email sponsored by: Free pre-built ASP.NET sites including
Data Reports, E-commerce, Portals, and Forums are available now.
Download today and enter to win an XBOX or Visual Studio .NET.
http://aspnet.click-url.com/go/psa00100003ave/direct;at.aspnet_072303_01/01