Memory Leaks in BooleanFunction

[pfasante]

As discussed in #14549 there is a memory leak in BooleanPolynomialRing, which was fixed (somewhat hacky) for the annihilator method by using BooleanPolynomialRing_constructor.

There are other uses of BooleanPolyonmialRing which should be fixed, too. I suggest to do this the same way as for annihilator. But maybe also add a comment to the code, to link to this and #14549 discussion, so we have a later reference, why these changes exist.

Component: memleak

Keywords: BooleanFunction

Reviewer: Travis Scrimshaw

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

[pfasante]

Branch: u/asante/memory_leaks_in_booleanfunction

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

b29c0e5

add a comment to remember that this is only a hack

[sagetrac-git]

Commit: b29c0e5

[pfasante]

comment:5

(sorry forgott that I still have to fix something)

[sagetrac-git]

Branch pushed to git repo; I updated commit sha1. New commits:

90f0a83

import the factory function from the right module