<p dir="ltr">I think non-working is a misnomer here. These do not break the build.</p>
<p dir="ltr">    Matt</p>
<div class="gmail_quote">On Feb 2, 2013 4:23 PM, "Jed Brown" <<a href="mailto:jedbrown@mcs.anl.gov">jedbrown@mcs.anl.gov</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div><font face="arial, helvetica, sans-serif">It seems like every time I pull PETSc, I get more build noise from unused or uninitialized variables. I have a suggestion: if it's not clean yet, just don't push it. If you want a backup or to communicate the messy code with someone else, push it to <a href="http://bitbucket.org/YOURNAME/petsc-dev" target="_blank">bitbucket.org/YOURNAME/petsc-dev</a>. Pushing to petsc-dev should mean that it's either ready for users or that it's at least ready for other people to start working with.</font></div>

<div><font face="arial, helvetica, sans-serif"><br></font></div><div><font face="arial, helvetica, sans-serif">Pushing obviously broken code to petsc-dev discourages people from reading the commit logs because it's exhausting and it's hard to argue with "yeah, I know none of that works" (except to fix the process). Lack of review lowers the quality of code in PETSc.</font></div>

<div><font face="arial, helvetica, sans-serif"><br></font></div><div><font face="arial, helvetica, sans-serif"><br></font></div><div><font face="arial, helvetica, sans-serif">It's Matt's code today, but we've all been guilty on occasion.<br>

</font><br><font face="courier new, monospace">/home/jed/petsc/src/dm/impls/plex/plex.c:10062:19: warning: unused variable 'numSubPoints' [-Wunused-variable]</font></div><div><font face="courier new, monospace">  PetscInt       *numSubPoints, *coneNew;</font></div>

<div><font face="courier new, monospace">                  ^</font></div><div><font face="courier new, monospace">/home/jed/petsc/src/dm/impls/plex/plex.c[ 68%] :10209:7: warning: variable 'coneNew' is uninitialized when used here [-Wuninitialized]</font></div>

<div><font face="courier new, monospace">      coneNew[c] = firstSubVertex + subvertex;</font></div><div><font face="courier new, monospace">      ^~~~~~~</font></div><div><font face="courier new, monospace">/home/jed/petsc/src/dm/impls/plex/plex.c:10062:41: note: initialize the variable 'coneNew' to silence this warning</font></div>

<div><font face="courier new, monospace">  PetscInt       *numSubPoints, *coneNew;</font></div><div><font face="courier new, monospace">                                        ^</font></div><div><font face="courier new, monospace">                                         = NULL</font></div>

<div><font face="courier new, monospace">/home/jed/petsc/src/dm/impls/plex/plex.c:10280:38: warning: variable 'subpointMap' is uninitialized when used here [-Wuninitialized]</font></div><div><font face="courier new, monospace">  ierr = DMPlexSetSubpointMap(subdm, subpointMap);CHKERRQ(ierr);</font></div>

<div><font face="courier new, monospace">                                     ^~~~~~~~~~~</font></div><div><font face="courier new, monospace">/home/jed/petsc/src/dm/impls/plex/plex.c:10060:76: note: initialize the variable 'subpointMap' to silence this warning</font></div>

<div><font face="courier new, monospace">  IS              subvertexIS, subedgeIS, subfaceIS, subcellIS, subpointMap;</font></div><div><font face="courier new, monospace">                                                                           ^</font></div>

<div><font face="courier new, monospace">                                                                            = NULL</font></div><div><br></div></div>
</blockquote></div>