Bug 24457 - NPE for @see tag
Summary: NPE for @see tag
Status: RESOLVED FIXED
Alias: None
Product: classpath
Classification: Unclassified
Component: gjdoc (show other bugs)
Version: unspecified
: P2 normal
Target Milestone: ---
Assignee: Julian Scheid
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-10-20 17:41 UTC by Stephan Michels
Modified: 2005-10-20 18:31 UTC (History)
1 user (show)

See Also:
Host:
Target:
Build:
Known to work:
Known to fail:
Last reconfirmed:


Attachments
Simple java class to reproduce the NPE (147 bytes, text/plain)
2005-10-20 17:43 UTC, Stephan Michels
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Stephan Michels 2005-10-20 17:41:08 UTC
Hi,
I currently build the javadocs of Eclipse with gjdoc. But I got a NPE for some classes. I managed to find the exact position of the failure.

java.lang.NullPointerException
        at gnu.classpath.tools.doclets.htmldoclet.HtmlDoclet.renderSeeTag(HtmlDoclet.java:3572)
        at gnu.classpath.tools.doclets.htmldoclet.HtmlDoclet.renderTag(HtmlDoclet.java:3675)
        at gnu.classpath.tools.doclets.AbstractDoclet.printTaglets(AbstractDoclet.java:721)
        at gnu.classpath.tools.doclets.AbstractDoclet.printMainTaglets(AbstractDoclet.java:758)
        at gnu.classpath.tools.doclets.htmldoclet.HtmlDoclet.printTaglets(HtmlDoclet.java:2930)
        at gnu.classpath.tools.doclets.htmldoclet.HtmlDoclet.printMemberDetails(HtmlDoclet.java:2690)
        at gnu.classpath.tools.doclets.htmldoclet.HtmlDoclet.printClassPage(HtmlDoclet.java:2285)
        at gnu.classpath.tools.doclets.htmldoclet.HtmlDoclet.run(HtmlDoclet.java:3154)
        at gnu.classpath.tools.doclets.AbstractDoclet.startInstance(AbstractDoclet.java:200)
        at gnu.classpath.tools.doclets.AbstractDoclet.start(AbstractDoclet.java:131)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:85)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:58)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:60)
        at java.lang.reflect.Method.invoke(Method.java:391)
        at gnu.classpath.tools.gjdoc.Main.startDoclet(Main.java:620)
        at gnu.classpath.tools.gjdoc.Main.start(Main.java:1149)
        at gnu.classpath.tools.gjdoc.Main.main(Main.java:873)
Comment 1 Stephan Michels 2005-10-20 17:43:29 UTC
Created attachment 10033 [details]
Simple java class to reproduce the NPE

The exception only occur if you use a "link" argument, like "gjdoc -link http://java.sun.com/j2se/1.4.2/docs/api Layout.java"
Comment 2 Andrew Haley 2005-10-20 17:50:17 UTC
The error is here:

            if ((null == referencedPackageName
                 || packageName.length() > referencedPackageName.length())
                && referencedClassName.startsWith(packageName + '.')) {
               referencedPackageName = packageName;

Both are null, referencedClassName and packageName
Comment 3 cvs-commit@developer.classpath.org 2005-10-20 18:27:20 UTC
Subject: Bug 24457

CVSROOT:	/cvsroot/classpath
Module name:	gjdoc
Branch: 	
Changes by:	Julian Scheid <julian@savannah.gnu.org>	05/10/20 18:26:24

Modified files:
	src/gnu/classpath/tools/doclets/htmldoclet: HtmlDoclet.java 
	.              : ChangeLog 

Log message:
	2005-10-20  Julian Scheid  <julian@sektor37.de>
	
	PR gjdoc/24457:
	* src/gnu/classpath/tools/doclets/htmldoclet/HtmlDoclet.java
	(renderSeeTag): Fix NPE when see tag doesn't reference a class or
	member.

CVSWeb URLs:
http://savannah.gnu.org/cgi-bin/viewcvs/classpath/gjdoc/src/gnu/classpath/tools/doclets/htmldoclet/HtmlDoclet.java.diff?tr1=1.59&tr2=1.60&r1=text&r2=text
http://savannah.gnu.org/cgi-bin/viewcvs/classpath/gjdoc/ChangeLog.diff?tr1=1.231&tr2=1.232&r1=text&r2=text



_______________________________________________
Cp-tools-commit mailing list
Cp-tools-commit@gnu.org
http://lists.gnu.org/mailman/listinfo/cp-tools-commit

Comment 4 Julian Scheid 2005-10-20 18:31:47 UTC
This is fixed now.  Thanks for your report, Stephan.