DO NOT REPLY [Bug 40819] - NPE in IdResolver

[bu...@apache.org]

DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG�
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=40819>.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND�
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=40819





------- Additional Comments From jorge.nevado@ericsson.com  2007-02-05 03:51 -------
(In reply to comment #3)
> You must always use *NS methods, The non *NS methods should dia a horrible 
death.
> Also the code that is throwing the exception has not changed so I suppose it
> also happens in 1.4.beta2 and even 1.3.
> I can prevent the NPE but it will silently fail and will never find a element.
> The only sensible thing is to throw an exception when localName  is null (i.e.
> has be done with non NS member). 
> The problem is that then we should fill the code in several other parts with
> this check, to be consistent.

Why don�t you use both Attr APIs (i.e. getLocalName and getName)? This way, if 
getLocalName returns null for DOM1 cases or non qualified attributes, you could 
use getName for backward compatibility.

Greetings,
Jorge.
> What do you all think?
> Anyway I downgrade the severity of the bug. 



-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.