> My question is: to debug this, should the constructor be changed to > try to reasonably find the zero element of the base ring (from "pdict" > list), or should it be that whenever PolyDict object is constructed, > the "zero" parameter is set (thus it is no longer a optional > parameter)?
I have no strong preference but I guess the zero element should be made non- optional. This way the developer does not have to be careful to coerce all elements of pdict to the right type but one element is sufficient. It is an 'internal' class anyway, so it isn't that critical as long as it is properly documented. Cheers, Martin -- name: Martin Albrecht _pgp: http://pgp.mit.edu:11371/pks/lookup?op=get&search=0x8EF0DC99 _otr: 47F43D1A 5D68C36F 468BAEBA 640E8856 D7951CCF _www: http://www.informatik.uni-bremen.de/~malb _jab: [email protected] --~--~---------~--~----~------------~-------~--~----~ To post to this group, send an email to [email protected] To unsubscribe from this group, send an email to [email protected] For more options, visit this group at http://groups.google.com/group/sage-devel URL: http://www.sagemath.org -~----------~----~----~----~------~----~------~--~---
