Opened 7 years ago

Closed 7 years ago

#21140 closed enhancement (fixed)

Remove redundant _lmul_ and _rmul_ methods

Reported by: jdemeyer Owned by:
Priority: major Milestone: sage-7.3
Component: coercion Keywords:
Cc: Merged in:
Authors: Jeroen Demeyer Reviewers: Vincent Delecroix
Report Upstream: N/A Work issues:
Branch: 69d18c7 (Commits, GitHub, GitLab) Commit: 69d18c75875c8a9f770e9ef493d6e721854f4448
Dependencies: Stopgaps:

GitHub link to the corresponding issue

Description (last modified by jdemeyer)

  1. Assume commutativity by default and implement _rmul_ as falling back to _lmul_ (like we already do for AdditiveGroupElement).
  1. Implement the basic _lmul_ and _rmul_ just once in ModuleElement: remove the redundant implementations from RingElement and AdditiveGroupElement.

Change History (6)

comment:1 Changed 7 years ago by jdemeyer

Description: modified (diff)
Summary: Remove redundant _lmul_ and _rmul_Remove redundant _lmul_ and _rmul_ methods

comment:2 Changed 7 years ago by jdemeyer

Branch: u/jdemeyer/remove_redundant__lmul__and__rmul__methods

comment:3 Changed 7 years ago by git

Commit: 69d18c75875c8a9f770e9ef493d6e721854f4448

Branch pushed to git repo; I updated commit sha1. This was a forced push. New commits:

69d18c7Remove redundant _lmul_ and _rmul_ methods

comment:4 Changed 7 years ago by jdemeyer

Status: newneeds_review

comment:5 Changed 7 years ago by vdelecroix

Reviewers: Vincent Delecroix
Status: needs_reviewpositive_review

comment:6 Changed 7 years ago by vbraun

Branch: u/jdemeyer/remove_redundant__lmul__and__rmul__methods69d18c75875c8a9f770e9ef493d6e721854f4448
Resolution: fixed
Status: positive_reviewclosed
Note: See TracTickets for help on using tickets.