Peter,

An additional enhacement that would be very very nice,
is to allow having a body-element. So I can write
something like this:

  <macrodef name="x" body="y">
    <sequential>
      <javac ...>
       <y/>
      </javac>
    </sequential>
  </macrodef>

And use it like:

 <x>
  <fileset..../>
  <classpath .../>
 </x>

instead of

 <x>
  <y>
   <fileset..../>
   <classpath .../>
  </y>
 </x>

For many macros, those that wrap some group of tasks
for example, it looks very odd to have to have an element
that covers all the tasks inside.

How dificult would be to achieve that?

Jose Alberto

> -----Original Message-----
> From: Peter Reilly [mailto:[EMAIL PROTECTED] 
> Sent: 08 January 2004 17:42
> To: Ant Developers List
> Subject: Re: cvs commit: ant/docs/manual/CoreTasks macrodef.html
> 
> 
> description attributes are allowed on tasks and targets, so 
> it is a logical extension to have them for the nested 
> elements - esp for macrodef 
> attributes/elements.
> Peter
> Dominique Devienne wrote:
> 
> >>From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]
> >>  Log:
> >>  Add description attributes for elements and attributes of macrodef
> >>  PR: 24711
> >>  Obtained from: gudnabrsam at yahoo dot com and Yuji Yamano
> >>    
> >>
> >
> >How about a addText(String) so that any text within the 
> attribute could 
> >be appended or replace the description attribute?
> >
> >Or allow a nested <description> element? Wait, isn't there this some 
> >rule that any <description> element is part of the build element or 
> >something!?!?
> >
> >I allows feel that XML attributes are unsuitable to hold 
> sometimes long 
> >text such as descriptions!
> >
> >--DD
> >
> >---------------------------------------------------------------------
> >To unsubscribe, e-mail: [EMAIL PROTECTED]
> >For additional commands, e-mail: [EMAIL PROTECTED]
> >
> >
> >
> >  
> >
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
> 
> 

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to