[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: SQL backend C source error (ITS#2309)



Pierangelo Masarati wrote:
>>File: ./servers/slapd/back-sql/init.c, line 34
>>The problem is this name doesn't exist. I believe the correct line should be:
>>
>>bi.bi_init = sql_back_initialize;
> 
> 
> I guess this fix is correct; I never tried the module version.

Yeah, not even GNU CC would swallow it :-)

> This was fine with my compilers; however, there's no reason 
> not to accept this change, either.
> 
> Applied to HEAD

Well, I'm not sure what standard of C allows for structures to be statically 
initialized. The construct "{ AAA, BBB }" is for static initialization of 
arrays. Now, if I wanted to extend C specification to allow for structs to be 
initialized statically, then that is what I's use or "( AAA, BBB )", but that 
resembles function call too much.

I suppose GNU CC allows that, just as it allows indented CPP directives (a 
common source of headaches when compiling with DEC CC and the first thing I 
cleanup in any source package). I would like not to be arrogant, demanding for 
"C correctness", but is there a standard? (Yes, *several* of them :-))

Thank you for being thoughtful of us who do not use C compilers with extended 
syntax and making the source more compliant.

Nix.