<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><br class=""><div> I like the diversity of slightly different test boxes. If all test boxes are the same than that limits what variants we test and reduces the usefulness of the CI; since users will face this diversity in their machines and end up with errors we never see before hand (which is bad).</div><div><br class=""></div><div> This kind of "after the fact initialization "for (q = 0; q < LANDAU_MAX_Q_FACE; q++) col_scale[q] = 0.0; " is unlikely to satisfy the various compilers and optimization levels. Best to figure out universally how to indicate when the variable is declared that it gets initialized suitably (for confused compilers that don't realize the initialized values are never used and so print warning messages).</div><div><br class=""></div><div><br class=""></div><div> This seems to work for C++? <a href="https://icarus.cs.weber.edu/~dab/cs1410/textbook/7.Arrays/initialize.html" class="">https://icarus.cs.weber.edu/~dab/cs1410/textbook/7.Arrays/initialize.html</a></div><div><br class=""></div><div> <a href="https://stackoverflow.com/questions/201101/how-to-initialize-all-members-of-an-array-to-the-same-value" class="">https://stackoverflow.com/questions/201101/how-to-initialize-all-members-of-an-array-to-the-same-value</a> seems to indicate one can initialize all entries in C with one {0} but who trusts the web or all compilers.</div><div><br class=""></div><div> Have you tried the C form?</div><div><br class=""></div><div>Barry</div><div><br class=""></div><div><br class=""></div><div><br class=""></div><div><br class=""><blockquote type="cite" class=""><div class="">On Apr 12, 2021, at 8:06 AM, Satish Balay via petsc-dev <<a href="mailto:petsc-dev@mcs.anl.gov" class="">petsc-dev@mcs.anl.gov</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">This job gets run on either pj01 or pj02 - each have slightly different version of gcc. (and they behave differently wrt certain warnings)<br class=""><br class="">[trade-off with diversity in testing and having redundancy in available work machines - so jobs are not stuck on a single box]<br class=""><br class="">I guess I should somehow fix this (perhaps switch them all to same OS/versions].<br class=""><br class="">Satish<br class=""><br class="">On Sun, 11 Apr 2021, Mark Adams wrote:<br class=""><br class=""><blockquote type="cite" class="">This warning went away. i don't know why.<br class=""><br class="">On Sun, Apr 11, 2021 at 2:10 PM Mark Adams <<a href="mailto:mfadams@lbl.gov" class="">mfadams@lbl.gov</a>> wrote:<br class=""><br class=""><blockquote type="cite" class="">I get this error in CI with complex float:<br class=""><a href="https://gitlab.com/petsc/petsc/-/jobs/1170144554" class="">https://gitlab.com/petsc/petsc/-/jobs/1170144554</a><br class=""><br class="">I put in code earlier to fix this warning about uninitialized vars, but<br class="">now it does not seem to work:<br class=""><br class="">/home/glci/builds-stage1/AbTGp5-t/0/petsc/petsc/src/ts/utils/dmplexlandau/plexland.c:414:94:<br class="">error: ‘col_scale[4]’ may be used uninitialized in this function<br class="">[-Werror=maybe-uninitialized]<br class="">2251 <https://gitlab.com/petsc/petsc/-/jobs/1170144554#L2251> PetscScalar<br class="">vals[LANDAU_MAX_Q_FACE*LANDAU_MAX_Q_FACE],row_scale[LANDAU_MAX_Q_FACE],col_scale[LANDAU_MAX_Q_FACE];<br class=""><br class="">Yet I have code to initialize col_scale:<br class=""><br class=""> for (q = 0; q < LANDAU_MAX_Q_FACE; q++) col_scale[q] = 0.0; //<br class="">suppress warnings<br class=""><br class="">I get a similar error with an integer array.<br class=""><br class="">Any suggestions?<br class=""><br class=""></blockquote><br class=""></blockquote></div></div></blockquote></div><br class=""></body></html>