Changes between Version 1 and Version 2 of Ticket #5457, comment 54


Ignore:
Timestamp:
08/11/12 12:40:01 (8 years ago)
Author:
bump
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #5457, comment 54

    v1 v2  
    66
    77This patch removes the {{{__cmp__}}} method from llt polynomials instead
    8 of adding one for sf. I started to change to positive review, but I think
    9 I'll leave it as needs_review until Nicolas has looked at the patch too.
     8of adding one for sf. In fact, {{{SymmetricFunctions}}} should not have had a
     9{{{__cmp__}}} method because it already inherits one from {{{UniqueRepresentation}}}.
     10(Tested!) On the other hand, there is no reason for the test as previously written
     11to be deterministic. So this is the correct approach.