[petsc-dev] Please stop pushing code with warnings

Jed Brown jedbrown at mcs.anl.gov
Sat Nov 3 12:29:45 CDT 2012


This noise in the build output is ugly and unprofessional. If something
isn't ready yet, just don't push it. You can continue to rebase against
upstream until the feature is ready to be looked at by others.

/home/jed/petsc/src/mat/impls/nest/matnest.c: In function
‘MatConvert_Nest_AIJ’:
/home/jed/petsc/src/mat/impls/nest/matnest.c:1416:30: warning: variable
‘bmonnz’ set but not used [-Wunused-but-set-variable]

/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c: In function
‘MatMatMatMult_SeqAIJ_SeqAIJ_SeqAIJ’:
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:17:18: warning:
unused variable ‘ierr’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c: In function
‘MatMatMatMultSymbolic_SeqAIJ_SeqAIJ_SeqAIJ’:
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:45:44: warning:
unused variable ‘current_space’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:45:22: warning:
unused variable ‘free_space’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:44:22: warning:
unused variable ‘lnkbt’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:43:68: warning:
unused variable ‘ndouble’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:43:64: warning:
unused variable ‘lnk’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:43:54: warning:
unused variable ‘nlnk_max’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:43:51: warning:
unused variable ‘aj’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:43:47: warning:
unused variable ‘bj’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:43:41: warning:
unused variable ‘cnzi’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:43:36: warning:
unused variable ‘bnzj’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:43:31: warning:
unused variable ‘brow’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:43:26: warning:
unused variable ‘anzi’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:43:24: warning:
unused variable ‘j’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:43:22: warning:
unused variable ‘i’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:42:22: warning:
unused variable ‘afill’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:41:50: warning:
unused variable ‘bm’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:41:36: warning:
unused variable ‘bn’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:41:22: warning:
unused variable ‘am’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:40:45: warning:
unused variable ‘cj’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:40:41: warning:
unused variable ‘ci’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:40:32: warning:
unused variable ‘bi’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:40:23: warning:
unused variable ‘ai’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:39:71: warning:
unused variable ‘c’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:38:22: warning:
unused variable ‘ierr’ [-Wunused-variable]
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c: In function
‘MatMatMatMultNumeric_SeqAIJ_SeqAIJ_SeqAIJ’:
/home/jed/petsc/src/mat/impls/aij/seq/matmatmatmult.c:56:18: warning:
unused variable ‘ierr’ [-Wunused-variable]

/home/jed/petsc/src/dm/impls/patch/patch.c: In function ‘DMPatchZoom’:
/home/jed/petsc/src/dm/impls/patch/patch.c:45:19: warning: unused variable
‘ierr’ [-Wunused-variable]
/home/jed/petsc/src/dm/impls/patch/patch.c:44:19: warning: unused variable
‘st’ [-Wunused-variable]
/home/jed/petsc/src/dm/impls/patch/patch.c:43:29: warning: unused variable
‘sw’ [-Wunused-variable]
/home/jed/petsc/src/dm/impls/patch/patch.c:43:24: warning: unused variable
‘dof’ [-Wunused-variable]
/home/jed/petsc/src/dm/impls/patch/patch.c:43:19: warning: unused variable
‘dim’ [-Wunused-variable]
/home/jed/petsc/src/dm/impls/patch/patch.c: In function ‘DMPatchSolve’:
/home/jed/petsc/src/dm/impls/patch/patch.c:82:18: warning: unused variable
‘ierr’ [-Wunused-variable]
/home/jed/petsc/src/dm/impls/patch/patch.c:81:18: warning: unused variable
‘size’ [-Wunused-variable]
/home/jed/petsc/src/dm/impls/patch/patch.c:80:18: warning: unused variable
‘comm’ [-Wunused-variable]
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mcs.anl.gov/pipermail/petsc-dev/attachments/20121103/a1d5a68c/attachment.html>


More information about the petsc-dev mailing list