Project

Profile

Help

Actions
Send by e-mail Copy link

Bug #2806

closed

Incorrect code in FLWORExpression.copy()

Added by Michael Kay almost 8 years ago. Updated almost 7 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Michael Kay
Category:
Internals
Sprint/Milestone:
-
Start date:
2016-06-21
Due date:
% Done:

100%

Estimated time:
Legacy ID:
Applies to branch:
9.7, trunk
Fix Committed on Branch:
9.7, trunk
Fixed in Maintenance Release:
9.7.0.7
Platforms:

Description

In FLWORExpression.copy, the new clauses are linked to the old FLWORExpression rather than to the new FLWORExpression, leading to an incorrect parent pointer in the expression tree.

I've fixed this for 9.8 (where it caused a failure in the new loop lifting code) but I'm not currently aware of any symptoms for 9.7.

Actions
Copy link
 #1

Updated by Michael Kay almost 8 years ago

  • Category set to Internals
  • Status changed from New to Resolved
  • Assignee set to Michael Kay
  • Priority changed from Low to Normal
  • Applies to branch 9.7, 9.8 added
  • Fix Committed on Branch 9.7, 9.8 added

I have corrected the code on the 9.7 branch and confirmed that the change causes no regression to QT3 tests.

Actions
Copy link
 #2

Updated by O'Neil Delpratt almost 8 years ago

  • % Done changed from 0 to 100
  • Fixed in Maintenance Release 9.7.0.7 added

Bug fix applied in the Saxon 9.7.0.7 maintenance release.

Actions
Copy link
 #3

Updated by O'Neil Delpratt almost 8 years ago

  • Status changed from Resolved to Closed
Actions
Copy link
 #4

Updated by O'Neil Delpratt almost 7 years ago

  • Applies to branch trunk added
  • Applies to branch deleted (9.8)
Actions
Copy link
 #5

Updated by O'Neil Delpratt almost 7 years ago

  • Fix Committed on Branch trunk added
  • Fix Committed on Branch deleted (9.8)

Please register to edit this issue

Actions
Send by e-mail Copy link

Also available in: Atom PDF