Remove wrapper_parent.pyx

[jdemeyer]

The file src/sage/structure/wrapper_parent.pyx is nowhere used in Sage, not doctested, and hasn't seen any non-trivial change since 2008. Moreover, there is almost no actual code, so even if it's not broken, it doesn't do much. Just remove it.

Component: coercion

Author: Jeroen Demeyer

Branch/Commit: 307c237

Reviewer: Marc Mezzarobba

Issue created by migration from https://trac.sagemath.org/ticket/17738

[jdemeyer]

Branch: u/jdemeyer/ticket/17738

[jdemeyer]

New commits:

307c237

Remove wrapper_parent

[jdemeyer]

Commit: 307c237

[mezzarobba]

comment:3

Why? Is it broken? Does it have a significant maintenance cost? Is there a better mechanism to achieve what it was intended for?

[jdemeyer]

comment:4

Replying to @mezzarobba:

Is it broken?

Given that it's not used, not doctested, and not maintained since 2008, it is very likely broken. Moreover, there is almost no actual code, so even if it's not broken, it doesn't do much.

Does it have a significant maintenance cost?

Not significant, but there is some maintanance cost: every time we deprecate or change something that this module uses, we have to update it.

Is there a better mechanism to achieve what it was intended for?

Your question seems to assume that this module itself achieves what it was intended for. I don't know if that's the case. Given the little code, I doubt it. In any case, I would need to understand better what it was intented for...

[jdemeyer]

comment:5

Let me also add that there is no discussion on Trac nor on sage-devel mentioning this module, so I very much doubt that it will be missed (and if so, it's still in the git history).

[mezzarobba]

Reviewer: Marc Mezzarobba

[mezzarobba]

comment:6

Sound convincing :-)

[vbraun]

Changed branch from u/jdemeyer/ticket/17738 to 307c237