Project

Profile

Help

Actions
Send by e-mail Copy link

Bug #2565

closed

Inefficient evaluation of exists()

Added by Michael Kay almost 9 years ago. Updated almost 9 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Michael Kay
Category:
Performance
Sprint/Milestone:
-
Start date:
2015-12-29
Due date:
% Done:

100%

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

Description

See https://saxonica.plan.io/boards/3/topics/6256

Also applies to empty() and perhaps other functions.

Actions
Copy link
 #1

Updated by Michael Kay almost 9 years ago

  • Status changed from In Progress to Resolved
  • Fix Committed on Branch 9.7 added

I have fixed exists() to work the way it did before. The code for empty() already had this optimization (and there was a comment in the code for exists() asking why it differed).

One thing this bug report reveals is that these little optimizations can sometimes make a surprisingly big difference to bottom-line performance.

Actions
Copy link
 #2

Updated by O'Neil Delpratt almost 9 years ago

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

Bug fix applied in the Saxon 9.7.0.2 maintenance release

Actions
Copy link
 #3

Updated by O'Neil Delpratt almost 9 years ago

  • Status changed from Resolved to Closed

Please register to edit this issue

Actions
Send by e-mail Copy link

Also available in: Atom PDF