You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@daffodil.apache.org by "Dave Thompson (Jira)" <ji...@apache.org> on 2021/11/08 13:02:00 UTC
[jira] [Closed] (DAFFODIL-2578) Raise minimum C level to ISO C11
with GNU extensions
[ https://issues.apache.org/jira/browse/DAFFODIL-2578?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Dave Thompson closed DAFFODIL-2578.
-----------------------------------
Verified the specified commit (commit 4fd08c65ccbc88334b4be010b4258bec61e1435f) is included in the latest pull from the daffodil repository.
Verified changes identified in commit comment were implemented.
Verified all affected daffodil sub-project sbt test suites execute successfully.
Verified the runtime2 tests now pass with loop variable i defined within for loop statements.
Verified the nightly test schemas compile and save successfully.
Verified the nightly test suite executes successfully.
> Raise minimum C level to ISO C11 with GNU extensions
> ----------------------------------------------------
>
> Key: DAFFODIL-2578
> URL: https://issues.apache.org/jira/browse/DAFFODIL-2578
> Project: Daffodil
> Issue Type: Bug
> Components: Back End C-Generator
> Reporter: John Interrante
> Priority: Minor
> Fix For: 3.2.0
>
>
> Fix pedantic warning "ISO C99 doesn't support unnamed structs/unions" found when compiling nested.dfdl.xsd's NestedUnion example with current Makefile's CFLAGS. Verify that both gcc and clang support "-std=gnu11" all the way back to gcc-4 and clang-3 on CentOS 7. Raise runtime2's minimum C language support level to ISO C11 with GNU extensions and make all the places where Daffodil calls the C compiler pass "-std=gnu11" to the C compiler to compile code with the same minimum C level. ISO C11 also allows "for (" loops to define loop variables in place so we can make that change too.
--
This message was sent by Atlassian Jira
(v8.20.1#820001)