<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 1, 2014 at 5:40 PM, Dave Goodell (dgoodell) <span dir="ltr"><<a href="mailto:dgoodell@cisco.com" target="_blank">dgoodell@cisco.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Begin forwarded message:<br>
<br>
> From: Service Account <<a href="mailto:noreply@mpich.org">noreply@mpich.org</a>><br>
> Subject: [mpich-commits] [mpich] MPICH primary repository branch, master, updated. v3.2a1-34-geda105b<br>
> Date: October 1, 2014 5:29:25 PM CDT<br>
> To: <<a href="mailto:all-commits@mpich.org">all-commits@mpich.org</a>>, <<a href="mailto:commits@mpich.org">commits@mpich.org</a>><br>
> Reply-To: <<a href="mailto:commits@mpich.org">commits@mpich.org</a>><br>
<div><div class="h5">><br>
> This is an automated email from the git hooks/post-receive script. It was<br>
> generated because a ref change was pushed to the repository containing<br>
> the project "MPICH primary repository".<br>
><br>
> The branch, master has been updated<br>
>       via  eda105b029572ebf7f4008f6de2e1afb9ccf6fc2 (commit)<br>
>       via  98114df4c0504568ab188ea98a9d1269d087bd82 (commit)<br>
>       via  ad25799865749680338c05264c8007242f01326a (commit)<br>
>      from  6b8c426d90b28bb0ed6ff6ade6ac03a4e4b96634 (commit)<br>
><br>
> Those revisions listed above that are new to this repository have<br>
> not appeared on any other notification email; so we list those<br>
> revisions in full, below.<br>
><br>
> - Log -----------------------------------------------------------------<br>
> <a href="http://git.mpich.org/mpich.git/commitdiff/eda105b029572ebf7f4008f6de2e1afb9ccf6fc2" target="_blank">http://git.mpich.org/mpich.git/commitdiff/eda105b029572ebf7f4008f6de2e1afb9ccf6fc2</a><br>
><br>
> commit eda105b029572ebf7f4008f6de2e1afb9ccf6fc2<br>
> Author: Junchao Zhang <<a href="mailto:jczhang@mcs.anl.gov">jczhang@mcs.anl.gov</a>><br>
> Date:   Tue Sep 16 15:58:55 2014 -0500<br>
><br>
>    Remove unnessary array init in MPIU_CHKLMEM_DECL<br>
><br>
>    Initializing mpiu_chklmem_stk_sp_ to 0 should already have the effect to<br>
>    initialize mpiu_chklmem_stk_[n_].<br>
><br>
>    Signed-off-by: Antonio J. Pena <<a href="mailto:apenya@mcs.anl.gov">apenya@mcs.anl.gov</a>><br>
><br>
> diff --git a/src/include/mpimem.h b/src/include/mpimem.h<br>
> index a217e8b..e1478a5 100644<br>
> --- a/src/include/mpimem.h<br>
> +++ b/src/include/mpimem.h<br>
> @@ -317,7 +317,7 @@ extern char *strdup( const char * );<br>
> }}<br>
> #else<br>
> #define MPIU_CHKLMEM_DECL(n_) \<br>
> - void *(mpiu_chklmem_stk_[n_]) = {0};\<br>
> + void *(mpiu_chklmem_stk_[n_]); \<br>
>  int mpiu_chklmem_stk_sp_=0;\<br>
>  MPIU_AssertDeclValue(const int mpiu_chklmem_stk_sz_,n_)<br>
<br>
</div></div>You might want to revert this.  My recollection is that not all compilers are smart enough to see that mpiu_chklmem_stk_ is not used before being defined, which can lead to a *lot* of warnings throughout the MPICH codebase.<br></blockquote><div><br></div><div>Dave, I don't quite understand your concern. I removed the array initialization code since MPIU_CHKLMEM_FREEALL() can only free pointers set before. The number of pointers is <span style="color:rgb(80,0,80)">mpiu_chklmem_stk_sp_, which is initialized to 0.</span></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
Also, what's the sudden interest in restoring C89 support again?  The whole world really does support enough of C99 now...  Seems like a big waste of time and an unnecessary annoyance to developers who might want to use useful C99 features (e.g., named structure initializers).<br> <br></blockquote><div>See <a href="https://trac.mpich.org/projects/mpich/ticket/2167">https://trac.mpich.org/projects/mpich/ticket/2167</a></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
-Dave<br>
<br>
_______________________________________________<br>
To manage subscription options or unsubscribe:<br>
<a href="https://lists.mpich.org/mailman/listinfo/devel" target="_blank">https://lists.mpich.org/mailman/listinfo/devel</a><br>
</blockquote></div><br></div></div>