Project

General

Profile

Design #491

Cleaning: Cruft in CoCoA-5 packages

Added by John Abbott about 4 years ago. Updated 6 months ago.

Status:
In Progress
Priority:
High
Assignee:
-
Category:
Cleaning
Target version:
Start date:
22 Mar 2014
Due date:
% Done:

20%

Estimated time:
Spent time:

Description

There is a HUGE amount of cruft in the C5 packages; more than enough to make maintenance much harder than necessary (I've been lowercasing some fn names).

Some packages seems to define their own versions of some "common" fns such as Den in hp.pkg5; moreover, I think this fn is sometimes called without the $. prefix -- this surely looks like a recipe for trouble! Indeed I wonder if my lowercasing of the packages has introduced some bugs; there's such a mess, it's almost impossible for me to tell!

Reduce the amount of cruft!
[I know most people will never look in the packages, but the few that do won't be impressed!]


Related issues

Related to CoCoA-5 - Design #493: Cleanup Hilbert package hp.cpkg5Closed2014-03-24

Related to CoCoA-5 - Support #940: Check all fns in packages work even without BackwardCompatible.cpkg5Closed2016-10-10

History

#1 Updated by John Abbott about 4 years ago

The packages that "upset" me the most are coclib and primary (or radical?)

#2 Updated by Anna Maria Bigatti about 4 years ago

  • Subject changed from Cruft in C5 packages to Cleaning: Cruft in C5 packages

#3 Updated by Anna Maria Bigatti about 4 years ago

  • Target version set to CoCoA-5.1.0 Easter14

#4 Updated by John Abbott about 4 years ago

  • Target version changed from CoCoA-5.1.0 Easter14 to CoCoA-5.1.1 Seoul14

#5 Updated by Anna Maria Bigatti over 3 years ago

This task will never be closed (when would we ever be able to say that? ;-)
So, let's say that we start deleting obsolete code when we see it....

#6 Updated by John Abbott over 3 years ago

  • Status changed from New to In Progress
  • Target version changed from CoCoA-5.1.1 Seoul14 to CoCoA-5.1.2 summer 2015
  • % Done changed from 0 to 10

Perhaps this task is best achieved by Anna and me together; we should meet occasionally and remove "dead" code from some packages.

Some of the cleaning may create a new redmine task to implement some feature in CoCoALib....

#7 Updated by John Abbott almost 3 years ago

  • Target version changed from CoCoA-5.1.2 summer 2015 to CoCoA-5.1.3/4 Jan 2016

#8 Updated by Anna Maria Bigatti about 2 years ago

  • Target version changed from CoCoA-5.1.3/4 Jan 2016 to CoCoA-5.2.0 spring 2017

#9 Updated by John Abbott over 1 year ago

  • Related to Support #940: Check all fns in packages work even without BackwardCompatible.cpkg5 added

#10 Updated by John Abbott over 1 year ago

  • Priority changed from Normal to High
  • Target version changed from CoCoA-5.2.0 spring 2017 to CoCoA-5.2.2
  • % Done changed from 10 to 20

This is going to be a long and tedious job; moreover it should probably be done when Anna and I can be together.

From a user's point of view it is not so important (since everything seems to work fine), but we should certainly do something before too long.

#11 Updated by Anna Maria Bigatti over 1 year ago

  • Subject changed from Cleaning: Cruft in C5 packages to Cleaning: Cruft in CoCoA-5 packages

This issue seems dormant, in reality every time we open a package we clean it up a little bit ;-)
(for example with #940)

#12 Updated by Redmine Admin 6 months ago

  • Target version changed from CoCoA-5.2.2 to CoCoA-5.2.4

Also available in: Atom PDF