> At first glance I can not see if it is wrong or not: it depends > on intent of original writer. I suspect that the intention was > to use this code only for sparse sets.
See the patch that changes Rep of BinaryTree in the other thread, I think the intention here was for this BinaryTree case, after both changes, new BinaryTree can use = from BinaryRecursiveAggregate. After all, BinaryRecursiveAggregate is only used by BinaryTreeCategory, and BinaryTreeCategory doesn't have SetAggregate. -- You received this message because you are subscribed to the Google Groups "FriCAS - computer algebra system" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To post to this group, send email to [email protected]. Visit this group at https://groups.google.com/group/fricas-devel. For more options, visit https://groups.google.com/d/optout.
