On Sun, 01 Oct 2017, Tomi Ollila wrote:
> On Sun, Oct 01 2017, Jani Nikula wrote:
>
>> On Sun, 01 Oct 2017, Tomi Ollila wrote:
>>>
>>> Good stuff. It just doesn't longer compile on older compilers (so some
>>> note on commit message should be added):
>>
>>
On Sun, Oct 01 2017, Jani Nikula wrote:
> On Sun, 01 Oct 2017, Tomi Ollila wrote:
>>
>> Good stuff. It just doesn't longer compile on older compilers (so some
>> note on commit message should be added):
>
> Does this on top fix it? I used the unnamed struct just for clarity,
On Sun, 01 Oct 2017, Tomi Ollila wrote:
> On Sun, Oct 01 2017, Jani Nikula wrote:
>
>> Several changes at once, just to not have to change the same lines
>> several times over:
>>
>> - Use designated initializers to initialize opt desc arrays.
>>
>> - Only initialize the
On Sun, Oct 01 2017, Jani Nikula wrote:
> Several changes at once, just to not have to change the same lines
> several times over:
>
> - Use designated initializers to initialize opt desc arrays.
>
> - Only initialize the needed fields.
>
> - Remove arg_id (short options) as unused.
>
> - Replace
Jani Nikula writes:
> Several changes at once, just to not have to change the same lines
> several times over:
the general approach here looks fine. I didn't see any blockers, but
I'll wait a few days before merging.
This probably causes some rebasing pain for other people,
Several changes at once, just to not have to change the same lines
several times over:
- Use designated initializers to initialize opt desc arrays.
- Only initialize the needed fields.
- Remove arg_id (short options) as unused.
- Replace opt_type and output_var with several type safe output