If that's the case, then I wonder why it doesn't complain in other areas of the 
code where we also use C99 syntax? Or is it perhaps "mostly" C99 compliant, but 
doesn't like that specific use-case?


On Aug 20, 2014, at 7:20 AM, Nathan Hjelm <hje...@lanl.gov> wrote:

> Really? That means PGI 2013 is NOT C99 compliant! Figures.
> 
> -Nathan
> 
> On Tue, Aug 19, 2014 at 10:48:48PM -0400, svn-commit-mai...@open-mpi.org 
> wrote:
>> Author: ggouaillardet (Gilles Gouaillardet)
>> Date: 2014-08-19 22:48:47 EDT (Tue, 19 Aug 2014)
>> New Revision: 32555
>> URL: https://svn.open-mpi.org/trac/ompi/changeset/32555
>> 
>> Log:
>> btl/scif: use safe syntax
>> 
>> PGI compilers 2013 and older do not support the following syntax :
>> mca_btl_scif_modex_t modex = {.port_id = mca_btl_scif_module.port_id};
>> so split it on two lines
>> 
>> cmr=v1.8.2:reviewer=hjelmn
>> 
>> Text files modified: 
>>   trunk/opal/mca/btl/scif/btl_scif_component.c |     3 ++-                   
>>                   
>>   1 files changed, 2 insertions(+), 1 deletions(-)
>> 
>> Modified: trunk/opal/mca/btl/scif/btl_scif_component.c
>> ==============================================================================
>> --- trunk/opal/mca/btl/scif/btl_scif_component.c     Tue Aug 19 18:34:49 
>> 2014        (r32554)
>> +++ trunk/opal/mca/btl/scif/btl_scif_component.c     2014-08-19 22:48:47 EDT 
>> (Tue, 19 Aug 2014)      (r32555)
>> @@ -208,7 +208,8 @@
>> 
>> static int mca_btl_scif_modex_send (void)
>> {
>> -    mca_btl_scif_modex_t modex = {.port_id = mca_btl_scif_module.port_id};
>> +    mca_btl_scif_modex_t modex;
>> +    modex.port_id = mca_btl_scif_module.port_id;
>> 
>>     return opal_modex_send (&mca_btl_scif_component.super.btl_version, 
>> &modex, sizeof (modex));
>> }
>> _______________________________________________
>> svn mailing list
>> s...@open-mpi.org
>> http://www.open-mpi.org/mailman/listinfo.cgi/svn
> _______________________________________________
> devel mailing list
> de...@open-mpi.org
> Subscription: http://www.open-mpi.org/mailman/listinfo.cgi/devel
> Link to this post: 
> http://www.open-mpi.org/community/lists/devel/2014/08/15667.php

Reply via email to