[#XALANJ-2307] [PATCH] remove some unused code

XalanJ2

[PATCH] remove some unused code

Created: 16/Jul/06 09:39 PM Updated: 09/Jan/08 03:48 AM

Return to search

Component/s:

None

Affects Version/s:

2.7

Fix Version/s:

None

Security Level:

No security risk; visible to anyone (Ordinary problems in Xalan projects. Anybody can view the issue.)

Time Tracking:

Not Specified

File Attachments:

	File Name	Date Attached	Attached By	Size
	remove_unusd.diff	2006-07-16 09:39 PM	Dave Brosius	4 kB

Xalan info:	PatchAvailable
Fix priority:	fp3
Reviewer:	Brian Minchau
Resolution Date:	17/Oct/06 04:22 PM

Description

« Hide

remove some code that isn't used.

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

[ Permlink | « Hide ]

Dave Brosius added a comment - 16/Jul/06 09:39 PM

some simple removal of code that isn't used.

Dave Brosius made changes - 16/Jul/06 09:39 PM

Field	Original Value	New Value
Attachment		remove_unusd.diff [ 12336974 ]

[ Permlink | « Hide ]

Brian Minchau added a comment - 17/Oct/06 06:43 AM

Per the JIRA meeting on Oct 16, 2005, Brian M. agreed to review the patch.
Some concerns about remove code like this: DTM dtm = xctxt.getDTM(node);
since although the variable dtm is never used, there may be side effects.

Brian Minchau made changes - 17/Oct/06 06:44 AM

reviewer

minchau@ca.ibm.com

[ Permlink | « Hide ]

Brian Minchau added a comment - 17/Oct/06 04:22 PM

This issue has a patch that is a duplicate (except for some minor shiting of lines of code) to that in xalanj-2208 which was recently applied to the code base.

This issue is resolved.

Brian Minchau made changes - 17/Oct/06 04:22 PM

Status	Open [ 1 ]	Resolved [ 5 ]
Resolution		Duplicate [ 3 ]

[ Permlink | « Hide ]

Brian Minchau added a comment - 11/Dec/07 03:16 PM

This issue is no longer relevant for 2.7.1, it is fixed in 2.7.1.
Changing the affected version to 2.7.

Brian Minchau made changes - 11/Dec/07 03:16 PM

Affects Version/s		2.7 [ 11080 ]
Affects Version/s	2.7.1 [ 10863 ]

Dave Brosius made changes - 09/Jan/08 03:48 AM

Status

Resolved [ 5 ]

Closed [ 6 ]