Log In - Home Page

Patch: Fix xpath() to return namespace definitions (fixes the issue with nested or repeated xpath())

Edit Patch - Move To Another CommitFest - Delete Patch

CommitFest 2014-12
Topic Bug Fixes
Patch Status Committed
Author Ali Akbar
Reviewers Peter Eisentraut
Committer Peter Eisentraut
Close Date 2015-01-07
Comments
Patch by apaan on 2014-06-04 01:44:43 PM: Initial version.
Review by ams on 2014-06-16 03:52:49 PM: Works fine, tweaked patch attached.
Comment by tgl on 2014-07-07 11:30:32 PM: A couple things that look questionable.
Patch by apaan on 2014-07-11 08:51:10 AM: updated patch, more question.
Patch by apaan on 2014-08-29 01:29:37 PM: patch v4 (reimplementation with other method)
Patch by apaan on 2014-10-07 02:29:12 AM: patch v5: fix code formatting inconsistencies and patch format
Review by petere on 2014-11-04 03:17:58 PM: review/status update
Patch by apaan on 2014-11-05 10:56:01 AM: revert to last version, added test case
Comment by apaan on 2014-11-05 02:59:52 PM: note on performance regression
Patch by apaan on 2014-11-26 01:06:09 AM: patch v7: rebase to master
Comment by michael-kun on 2014-12-15 03:20:00 AM: Patch needs more work, per the author, hence switching to returned with feedback.
Comment by apaan on 2014-12-15 04:12:37 AM: the v7 patch is good as it is. Will try to reduce performance regression in another patch.

Add Comment

Please log in to comment on this patch.