Project

Profile

Help

Bug #4657

closed

Ambiguous accumulator rules - should choose the last

Added by Michael Kay over 3 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
XSLT conformance
Sprint/Milestone:
-
Start date:
2020-07-27
Due date:
% Done:

100%

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

Description

Reported by David Maus on xsl-list, test case accumulator-081 added.

Saxon seems to be choosing the first accumulator rule when it should choose the second.

Actions #1

Updated by Michael Kay over 3 years ago

  • Status changed from New to In Progress

It should be distinguishing the two rules by sequence number, but both have sequence number 0 (and other ranking attributes equal) so it's essentially choosing at random - perhaps we've got another test somewhere where it randomly gets it right.

The code in XSLAccumulator.compileDeclaration() calls Mode.addRule() at line 208; setting position (the penultimate argument) to zero rather than to a value that increments for each rule.

Actions #2

Updated by Michael Kay over 3 years ago

  • Status changed from In Progress to Resolved
  • Applies to branch 10, 9.9, trunk added
  • Fix Committed on Branch 10, 9.9, trunk added
Actions #3

Updated by O'Neil Delpratt over 3 years ago

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

Bug fix applied in the Saxon 10.2 maintenance release.

Actions #4

Updated by O'Neil Delpratt over 3 years ago

  • Status changed from Resolved to Closed
  • Fixed in Maintenance Release 9.9.1.8 added

Bug fix applied on the Saxon 9.9.1.8 maintenance release.

Please register to edit this issue

Also available in: Atom PDF