View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0000918 | 1003.1(2013)/Issue7+TC1 | System Interfaces | public | 2015-02-04 22:17 | 2019-06-10 08:54 |
| Reporter | safinaskar | Assigned To | |||
| Priority | normal | Severity | Editorial | Type | Error |
| Status | Closed | Resolution | Accepted | ||
| Name | Askar Safin | ||||
| Organization | |||||
| User Reference | |||||
| Section | getaddrinfo | ||||
| Page Number | 2013 edition, p. 927 | ||||
| Line Number | 31262 | ||||
| Interp Status | --- | ||||
| Final Accepted Text | |||||
| Summary | 0000918: C99 forbids "struct addrinfo hints = {}" | ||||
| Description | - | ||||
| Desired Action | -31262 struct addrinfo hints = {}; +31262 struct addrinfo hints = {0}; | ||||
| Tags | tc2-2008 | ||||
|
|
In addition to the problem raised here, there are a couple of problems with the related text on Page 924 Line 31149, "In this hints structure every member other than ai_flags, ai_family, ai_socktype, and ai_protocol shall be set to zero or a null pointer." 1. This is intended to be a requirement on the application, and so should be worded as "The application shall ensure that ..." 2. It is not clear whether "every member" refers only to the standard members or includes any non-standard additional members. I.e. can applications just set the standard members explicitly and leave any non-standard additional members with indeterminate values, or do they have to use a form of initialisation (as in the example) that implicitly sets all non-standard additional members to zero/null? |
|
|
The proposed initializer causes a compiler warning (missing initializer) at strict settings (-Wall -Wextra) of gcc and clang. The compilers do not complain if a designated initializer is used, like: struct addrinfo hints = { .ai_flags = 0 }; The following four lines can be folded into the initializer as well: struct addrinfo hints = { .ai_family = AF_UNSPEC, .ai_socktype = SOCK_DGRAM, .ai_flags = AI_PASSIVE }; |
|
|
Missing initializer is just a warning, "hints = {0}" still confirms to C99. But "hints = {.ai_flags = 0}" is bad, for example, because it is not supported in C++, even in C++14 |
|
|
The warning mentioned in note 0002538 should be fixed as part of the resolution of gcc bug 53119 ( https://gcc.gnu.org/bugzilla/show_bug.cgi?id=53119 ). If it's not, this should be reported. {0} is the correct idiomatic "universal zero initializer" especially for types whose details are not specified, and compilers should not be discouraging its use. For example it's the only portable way to initialize mbstate_t. |
|
|
gcc 4.9.2 already doesn't report this warning (gcc 4.9.2-10 in Debian). (and dalias, please, put space after links, your link now points to non-existent bug number "53119)") |
|
|
Same bug in clang: http://llvm.org/bugs/show_bug.cgi?id=21689 |
|
|
An additional bug will be filed to address the issues raised in 0000918:0002535. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2015-02-04 22:17 | safinaskar | New Issue | |
| 2015-02-04 22:17 | safinaskar | Name | => Askar Safin |
| 2015-02-04 22:17 | safinaskar | Section | => getaddrinfo |
| 2015-02-04 22:17 | safinaskar | Page Number | => 2013 edition, p. 927 |
| 2015-02-04 22:17 | safinaskar | Line Number | => 31262 |
| 2015-02-05 09:59 | geoffclare | Note Added: 0002535 | |
| 2015-02-06 14:52 | jilles | Note Added: 0002538 | |
| 2015-02-06 17:57 | safinaskar | Note Added: 0002539 | |
| 2015-02-06 19:23 | dalias | Note Added: 0002540 | |
| 2015-02-06 23:51 | safinaskar | Note Added: 0002541 | |
| 2015-02-07 00:13 | safinaskar | Note Added: 0002542 | |
| 2015-04-20 08:47 | geoffclare | Relationship added | related to 0000934 |
| 2015-04-23 15:10 | rhansen | Note Edited: 0002540 | |
| 2015-04-23 16:30 | Don Cragun | Interp Status | => --- |
| 2015-04-23 16:30 | Don Cragun | Status | New => Resolved |
| 2015-04-23 16:30 | Don Cragun | Resolution | Open => Accepted |
| 2015-04-23 16:30 | Don Cragun | Tag Attached: tc2-2008 | |
| 2015-04-23 16:33 | Don Cragun | Note Added: 0002636 | |
| 2015-04-24 09:06 | geoffclare | Relationship added | related to 0000939 |
| 2019-06-10 08:54 | agadmin | Status | Resolved => Closed |
