#14656: slight QA with flint definition
---------------------------------+------------------------------------------
       Reporter:  fbissey        |         Owner:  tbd         
           Type:  defect         |        Status:  needs_review
       Priority:  major          |     Milestone:  sage-5.10   
      Component:  PLEASE CHANGE  |    Resolution:              
       Keywords:                 |   Work issues:              
Report Upstream:  N/A            |     Reviewers:              
        Authors:                 |     Merged in:              
   Dependencies:                 |      Stopgaps:              
---------------------------------+------------------------------------------
Changes (by fbissey):

  * status:  new => needs_review


Old description:

> When building sage in sage-on-gentoo we get the following QA report
> {{{
>  * QA Notice: Package triggers severe warnings which indicate that it
>  *            may exhibit random runtime failures.
>  * build/cythonized/sage/libs/flint/flint.c:664:3: warning: implicit
> declaration of function '_fmpz_cleanup_mpz_content' [-Wimplicit-function-
> declaration]
> }}}
> Which is quite correct as _fmpz_cleanup* functions are defined in
> flint/fmpz.h while the pxd file only includes flint/flint.h. It is
> possible that this file is included somehow, but it is cleaner just to
> add it.

New description:

 When building sage in sage-on-gentoo we get the following QA report
 {{{
  * QA Notice: Package triggers severe warnings which indicate that it
  *            may exhibit random runtime failures.
  * build/cythonized/sage/libs/flint/flint.c:664:3: warning: implicit
 declaration of function '_fmpz_cleanup_mpz_content' [-Wimplicit-function-
 declaration]
 }}}
 Which is quite correct as _fmpz_cleanup* functions are defined in
 flint/fmpz.h while the pxd file only includes flint/flint.h. It is
 possible that this file is included somehow, but it is cleaner just to add
 it.

 Apply to the sage library:
 * [attachment:trac_14656.patch]

--

Comment:

 With patch for review.

-- 
Ticket URL: <http://trac.sagemath.org/sage_trac/ticket/14656#comment:1>
Sage <http://www.sagemath.org>
Sage: Creating a Viable Open Source Alternative to Magma, Maple, Mathematica, 
and MATLAB

-- 
You received this message because you are subscribed to the Google Groups 
"sage-trac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sage-trac?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to