Merge pull request #296 from vneiger/fix_warnings_fglm

Fix compilation warnings [fglm]

Author: mohabsafey

src/fglm/Makefile.am

@@ -2,7 +2,7 @@ noinst_LTLIBRARIES 	= libfglm.la
 libfglm_la_SOURCES 	= fglm_core.c
 libfglm_ladir				=	$(includedir)/msolve/fglm
 libfglm_la_HEADERS	= fglm.h
-libfglm_la_CFLAGS		= $(SIMD_FLAGS) $(CPUEXT_FLAGS) $(OPENMP_CFLAGS)
+libfglm_la_CFLAGS		= $(SIMD_FLAGS) $(CPUEXT_FLAGS) $(OPENMP_CFLAGS) -Wall -Wextra
 
 EXTRA_DIST	=		fglm.h \
                 libfglm.h \

src/fglm/berlekamp_massey.c

@@ -191,8 +191,7 @@ void nmod_berlekamp_massey_add_point_modif(
     B->points->length = old_length + 1;
 }
 
-//shift ne sert pas 
-int nmod_em_gcd(nmod_berlekamp_massey_t B, long shift){
+int nmod_em_gcd(nmod_berlekamp_massey_t B){
   slong i, l, k, queue_len, queue_lo, queue_hi;
   queue_lo = B->npoints; // vaut 0 en entree
   queue_hi = B->points->length; //vaut 2*dim ou dim est la dimension du quotient
@@ -316,7 +315,6 @@ int nmod_em_gcd(nmod_berlekamp_massey_t B, long shift){
 }
 
 
-//shift ne sert pas 
 int nmod_em_gcd_preinstantiated(nmod_berlekamp_massey_t B, long shift){
   slong l, k, queue_len, queue_lo, queue_hi;
   queue_lo = B->npoints; // vaut 0 en entree
@@ -436,7 +434,6 @@ int nmod_em_gcd_preinstantiated(nmod_berlekamp_massey_t B, long shift){
   FLINT_ASSERT(2*nmod_poly_degree(B->V1) <= B->npoints);
   FLINT_ASSERT(2*nmod_poly_degree(B->R0) >= B->npoints);
   FLINT_ASSERT(2*nmod_poly_degree(B->R1) <  B->npoints);
-
 
   return 1;
 }

src/fglm/data_fglm.c

@@ -166,7 +166,7 @@ static inline param_t *allocate_fglm_param(mp_limb_t prime, long nvars){
 static inline void free_fglm_param(param_t *param){
   nmod_poly_clear(param->elim);
   nmod_poly_clear(param->denom);
-  for(szmat_t i = 0; i < param->nvars-1; i++){
+  for(szmat_t i = 0; i < (unsigned int)param->nvars-1; i++){
     nmod_poly_clear(param->coords[i]);
   }
   free(param->coords);