Re: [gobo-eiffel-develop] EDP and ET_C_GENERATOR

[Eric B. <er...@go...>]

Howard Thomson wrote:
> I have been bringing my EDP project up to date with respect to Gobo SVN, and 
> have identified a couple of bugs somewhere in ET_C_GENERATOR as of SVN 
> revision 6046 (I think).
> 
> The new implementation of once manifest strings has a bug, in that at least 
> one once manifest string has been misplaced by another one in EDP when 
> generated with the new ET_C_GENERATOR.

I will need more information about this, because just by looking at
the code I cannot see any bug. I will probably have to wait for your
SVN repository to be fixed, and then you will have to tell me
which once manifest string is wrong.

> There is also a C emitted code problem for an external C routine, where 
> previously the arguments were passed without casts, and are now cast to (char 
> *) which the C compiler complains about, although it does not alter (for a 
> 32-bit platform) the resulting code.

You are probably referring to these lines:

         -- When compiling with C++, MSVC++ does not want to convert
         -- 'void*' to non-'void*' implicitly.
     current_file.put_string ("(char*)")
     print_argument_name (l_name, current_file)

in class ET_C_GENERATOR in features `print_external_c_body' and
`print_external_cpp_body'. The explicit cast to char* is a quick
hack to make the C++ compiler happy (my C compiler did not complain
about this hack). Any more appropriate solution is welcome.

-- 
Eric Bezault
mailto:er...@go...
http://www.gobosoft.com