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

Commit

Permalink
Trac #32841: don't mention zn_poly in flint header comments.
Browse files Browse the repository at this point in the history
The flint_wrap.h and flint_ntl_wrap.h headers mention "zn_poly and
pari" as reasons for doing something specific. The zn_poly SPKG has
been removed, however; so we update those comments to mention only
pari.
  • Loading branch information
orlitzky authored and dimpase committed Dec 7, 2022
1 parent da17a80 commit 5507887
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
2 changes: 1 addition & 1 deletion src/sage/libs/flint/flint_ntl_wrap.h
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@

#include <gmp.h>

/* Save previous definition of ulong if any, as zn_poly and pari also use it */
/* Save previous definition of ulong if any, as pari also uses it */
#pragma push_macro("ulong")
#undef ulong

Expand Down
8 changes: 4 additions & 4 deletions src/sage/libs/flint/flint_wrap.h
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
#ifndef SAGE_FLINT_WRAP_H
#define SAGE_FLINT_WRAP_H
/* Using flint headers together in the same module as headers from some other
* libraries (zn_poly, pari, possibly others) as it defines the macros ulong
* and slong all over the place.
/* Using flint headers together in the same module as headers from
* some other libraries (pari, possibly others) as it defines the
* macros ulong and slong all over the place.
*
* What's worse is they are defined to types from GMP (mp_limb_t and
* mp_limb_signed_t respectively) which themselves can have system-dependent
Expand All @@ -16,7 +16,7 @@

#include <gmp.h>

/* Save previous definition of ulong if any, as zn_poly and pari also use it */
/* Save previous definition of ulong if any, as pari also uses it */
/* Should work on GCC, clang, MSVC */
#pragma push_macro("ulong")
#undef ulong
Expand Down

0 comments on commit 5507887

Please sign in to comment.