Skip to content
This repository has been archived by the owner on Apr 7, 2023. It is now read-only.

Silence compiler warnings #2

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

jamesjer
Copy link

These 4 commits each silence a class of compiler warnings. They are separate commits so they can be modified or removed from this pull request if you object to any of them.

@orlitzky
Copy link
Collaborator

Who's allowed to merge this and how do we get a new release? The build is going to break with upcoming compilers: https://bugs.gentoo.org/show_bug.cgi?id=898300

@orlitzky
Copy link
Collaborator

orlitzky commented Apr 6, 2023

Thanks for doing this, but I think it's time to give up on the separate CLI quadratic sieve. I also found some /tmp vulnerabilities at #3, but more importantly, a less-buggy version of the quadratic sieve is now available in FLINT itself. I've opened sagemath/sage#35419 to update Sage.

Once that goes in, I think we can retire this repository.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants