Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
RE: [aspectj-dev] ajdoc update

ajdoc is a lot closer to building aTrack now (thanks Mik!). I think I'm now running into the same issue George ran into with inner types, except that it causes a stack trace (probably because this is a doubly inner type!). Per Mik's request I've submitted a bugzilla report: https://bugs.eclipse.org/bugs/show_bug.cgi?id=57773

 

Ron

 

p.s. I'd suggest adding ajdoc and ajbrowser as components (as well as a development version) to Bugzilla.

 

 

------------Original Message------------

From: George Harley1 <GHARLEY@xxxxxxxxxx>

To: aspectj-dev@xxxxxxxxxxx

Date: Wed, Apr-7-2004 9:20 AM

Subject: RE: [aspectj-dev] ajdoc update


Hi Mik,

> I had to roll back some of Andy's fix to broken links:
> getRelativeComponent didn't work properly for internal anchor links (..html#<mumble>).


When I did a re-spin earlier today I found that ajdoc was producing HTML with broken links between advice in package A and advised members of types in package B.

Links between advice and advised types in the same package were OK.

The links to inner types were broken as well and the displayed text string for the links were also incorrect (e.g, if linking to packagefoo.ParentType.InnerType.membermethod() the visible text said it was linking to packagefoo.InnerType.membermethod()).

I sent a patch to Andy earlier. Let's see how it fares in the build.

Bye for now,
George
________________________________________
George C. Harley




"Mik Kersten" <beatmik@xxxxxxxxx>
Sent by: aspectj-dev-admin@xxxxxxxxxxx

07/04/2004 16:31

Please respond to
aspectj-dev

To
<aspectj-dev@xxxxxxxxxxx>
cc
Subject
RE: [aspectj-dev] ajdoc update






Based on Ron's feedback from running ajdoc on aTrack I've made the following
enhancements and a bug fix for dealing with declare forms.  Ron: please
Bugzilla so that this stuff gets recorded.  All: use the new -XajdocDebug
flag to send offending generated source files along with bug reports.

- Added support for 1.4 sources (use "-source 1.4" option)

- Made output class files go to documentation directory (-d) rather than
source directory.

- Added "-XajdocDebug" option that will keep the source files generated from
the ASM around in order to help people send better bug reports.

- Any declare forms that ajdoc doesn't know how to generate docs for get
printed to the console and indicated as a limitation.

I also extended test coverage to declare forms and added proper clearing of
state for multiple invocations from test cases.

Mik

> -----Original Message-----
> From: aspectj-dev-admin@xxxxxxxxxxx [mailto:aspectj-dev-admin@xxxxxxxxxxx]
> On Behalf Of Mik Kersten
> Sent: Tuesday, April 06, 2004 9:20 PM
> To: aspectj-dev@xxxxxxxxxxx
> Subject: [aspectj-dev] ajdoc update
>
> I just committed the updates to ajdoc needed for 1.2.  In a nutshell it's
> moved forward a ways from the previous ajdoc:
> - AJDT integration works
> - generates documentation for all the AspectJ modules
> - works on 1.3 and 1.4
> - comments and Javadoc tags are properly preserved and resolved
>
> Note that the hope is still for "declare" documentation to come in the
> form
> of a contribution.
>
> Getting it to work for our tree involved a bunch of bug fixes to deal with
> things like anonymous types.  To run from our tree in bootstrap mode use
> something like the attached batch script.  To run from AJDT point Eclipse
> at
> this command under the "Project -> Generate Javadoc" menu.  End users will
> be able to point at the "<aj-install-dir>/bin/ajdoc.bat".
>
> I had to roll back some of Andy's fix to broken links:
> getRelativeComponent
> didn't work properly for internal anchor links (..html#<mumble>).  Andy,
> could you check this against the current thing against whatever was
> breaking
> for you, and ideally add that code patter to CoverageTestCase?
>
> I added -XjavadocsInModel to make the addition of Javadoc strings to the
> ASM
> only happen when request it (prevent bloat of the model with strings.  The
> running time of ajdoc is now dominated by the compilation time.
>
> I had to make a change to a test case that I'm not quite comfortable with.
> AsmBuilderTest.testNullHandlingOfVisit() now checks for an NPE instead of
> an
> EmptyStackException.  I'm not sure how the original could have worked in
> the
> absence of the handle cache.
>
> Mik
>
> --
> http://kerstens.org/mik


_______________________________________________
aspectj-dev mailing list
aspectj-dev@xxxxxxxxxxx
http://dev.eclipse.org/mailman/listinfo/aspectj-dev


Back to the top