Project

General

Profile

Feature #244

Rings: default ctor & assignment

Added by John Abbott over 11 years ago. Updated about 11 years ago.

Status:
Closed
Priority:
Normal
Assignee:
John Abbott
Category:
Various
Target version:
CoCoALib-0.9952
Start date:
28 Sep 2012
Due date:
% Done:

100%

Estimated time:
Spent time:
1.25 h

Description

JAA proposes we add:
  • default ctor for ring (value is RingZZ())
  • assignment of rings

The second is certainly needed for the proposed changes to RingElem (see #221)

Now that we have a unique copy of RingZZ() we can use it in the default ctor.

Related issues

Related to CoCoALib - Feature #221: Better RingElemsClosed2012-08-08

Related to CoCoALib - Bug #864: Assignment of rings (esp. derived rings such as PolyRing)Closed2016-04-06

History

#1 Updated by Anna Maria Bigatti over 11 years ago

OK! As I said for #221

#2 Updated by John Abbott over 11 years ago

To enable assignment for ring I must enable assignment for SmartPtrIRC. I wonder what consequences that will have... Oh dear!

#3 Updated by John Abbott over 11 years ago

  • Status changed from New to In Progress
  • % Done changed from 0 to 40

A simple test example showed that assignment of rings was already working. A visual check of the source for SmartPtrIRC confirmed that assignment is already activated (despite the comment to the contrary).

I'll update the documentation (which says nothing at all about assignment).

#4 Updated by John Abbott over 11 years ago

  • % Done changed from 40 to 50

I have improved the doc for ring (adding assignment and equality tests).

I have also confirmed that the current CVS image does indeed allow assignment of rings.

#5 Updated by John Abbott about 11 years ago

  • Category set to Various
  • Status changed from In Progress to Closed
  • Assignee set to John Abbott
  • Target version set to CoCoALib-0.9952
  • % Done changed from 50 to 100

Since no problems have surfaced in the last 4 months, I'm closing this issue. The code and doc appear to be correctly updated.

#6 Updated by John Abbott about 8 years ago

  • Related to Bug #864: Assignment of rings (esp. derived rings such as PolyRing) added

Also available in: Atom PDF