Re: Javadoc patch

[Patrick Chanezon]

Hi Conor,
thanks for commiting that patch, my project will now be able to update to the
latest ant release.

As of distinguishing the attribute name of link and group from the nested
element name, I had no ideological opinion about it: it is just that in my
tests, it appeared that if they have the same name, an old build.xml file using
the attribute would not work anymore: you would get an exception.
And backward compatibility seems to me a very important requirement.

Did you test that scenario ?
If so I'm very happy with the solution.
If not I suggest you change the name of the nested elements.

P@

Conor MacNeill wrote:

> I have applied Patrick's Javadoc patch with a few changes.
>
> I change the nested elements from
>
> <groups> to <group>
> <links> to <link>
> <linkofflines> to <link offline="yes">
>
> This seems a little clearer to me since each element only describes a single
> link or group. I know Patrick wanted to distinguish the nested element from
> the attribute but I think it is still clear. Combining the two link types
> into a single element seemed logical and makes it easy to change from the
> offline form to the online form.
>
> I also added support for multiple source paths. To do this I added my
> PathTokenizer object which I have previously discussed.
>
> Let me know if there are any issues.
>
> Conor

--
Patrick Chanezon, iPlanet Market Maker- Portal/EServices Technical Lead
Netscape Communications Corp. - http://people.netscape.com/chanezon/
Opinions are my own.

"Two monks were arguing about a flag. One said: `The flag is moving.'
The other said: `The wind is moving.'
The sixth patriach happened to be passing by. He told them: `Not the wind, not
the flag; mind is moving.'"
Zen Koan


begin:vcard 
n:Chanezon;Patrick
tel;fax:603.388-9565
tel;work:650.937.6605
x-mozilla-html:TRUE
org:<CENTER><A HREF="http://www.iplanet.com/";><IMG SRC="http://www.iplanet.com/images/header_images/logo.gif"; BORDER=0 </A></CENTER>;Vortex
version:2.1
email;internet:[EMAIL PROTECTED]
title:<I>iPlanet Market Maker - Portal Technical Lead</I>
note:<A HREF="http://people.netscape.com/chanezon/";><IMG SRC="http://people.netscape.com/chanezon/patrick_photo_identite.jpg"; BORDER=0 HEIGHT=70 WIDTH=49></A><BR>Have Fun with <A HREF="http://www.javasoft.com/";><IMG SRC="http://www.javasoft.com/images/logos/javalogo52x88.gif"; border="0" HEIGHT="88" WIDTH="52"></A><BR><hr><BR>
adr;quoted-printable:;;Netscape Communications Corp.=0D=0A501 E. Middlefield Road=0D=0ABuilding 6=0D=0AMV-025;Mountain View;CA;94043;USA
fn:Patrick Chanezon
end:vcard