Code review comment for lp:~zorba-coders/zorba/fix_bug_1123843

Revision history for this message
Sorin Marian Nasoi (sorin.marian.nasoi) wrote :

> Why do you need the second deep-equal? (the one that reverses $x)
It's a fix for test case "fn-distinct-values-mixed-args-011" from "fn-distinct-values":

[...]
fn:distinct-values((xs:decimal('1.2'), '1.2'))
[...]
<assert-permutation>"1.2", 1.2</assert-permutation>
[...]

The proposed fix thus covers all existing usage of 'assert-permutation', not all possible cases that may exist.

« Back to merge proposal