[ http://nagoya.apache.org/jira/browse/XALANJ-666?page=history ]
Henry Zongaro updated XALANJ-666:
---------------------------------
Priority: Major
> SAXSourceLocator Not Reporting Location Correctly
> -------------------------------------------------
>
> Key: XALANJ-666
> URL: http://nagoya.apache.org/jira/browse/XALANJ-666
> Project: XalanJ2
> Type: Bug
> Components: Other
> Versions: 2.2.x
> Environment: Operating System: Windows NT/2K
> Platform: PC
> Reporter: Peter Bryant
> Assignee: Scott Boag
>
> In the constructor:
> public SAXSourceLocator(Locator locator)
> {
> m_locator = locator;
> this.setColumnNumber(locator.getColumnNumber());
> this.setLineNumber(locator.getLineNumber());
> this.setPublicId(locator.getPublicId());
> this.setSystemId(locator.getSystemId());
> }
> The correct values are being set.
> However, in the 'get' methods (e.g.)
> public int getColumnNumber()
> {
> return (null == m_locator) ? super.getColumnNumber() :
> m_locator.getColumnNumber();
> }
> it appears that values in m_locator have changed and no longer have the
> values
> they had upon initialisation.
> I would suggest that m_locator is removed from this class (and its values
> only
> unsed during construction). This would make this class immutable. And the
> get
> methods would simply be:
> public int getColumnNumber()
> {
> return super.getColumnNumber();
> }
> er, or even unneccesary (just rely on superclass implementation).
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://nagoya.apache.org/jira/secure/Administrators.jspa
-
If you want more information on JIRA, or have a bug to report see:
http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
