#5597: [with patch, needs review] rename coercion action methods
------------------------+---------------------------------------------------
Reporter: robertwb | Owner: robertwb
Type: defect | Status: needs_review
Priority: major | Milestone: sage-4.2
Component: coercion | Keywords:
Work_issues: | Author:
Reviewer: | Merged:
------------------------+---------------------------------------------------
Comment(by nthiery):
Replying to [comment:7 robertwb]:
> Yes, it should be trying all 4 of these options.
Ok. Then I would prefer:
a.act_on_left(b)
b.act_on_right(a)
a.acted_upon_right(b)
b.acted_upon_left(a)
which makes it easier to implement independently the left and right
actions on a module, and possibly override just one or the other in a
subclass.
That being said, we can leave things as is. Those modules for which left
and right action do not coincide can later implement a.acted_upon(...) by
delegating the work to
acted_upon_left and acted_upon_right.
--
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/5597#comment:8>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica,
and MATLAB
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups
"sage-trac" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to
[email protected]
For more options, visit this group at
http://groups.google.com/group/sage-trac?hl=en
-~----------~----~----~----~------~----~------~--~---
