From: Colin P. A. <co...@co...> - 2008-05-11 15:20:44
|
Eric, Can you do anything to stop these warning messages: gestalt_test_driver31.c:3409: warning: passing argument 1 of `GE_check_void' discards qualifiers from pointer target type I get a lot of them. -- Colin Adams Preston Lancashire |
From: Eric B. <er...@go...> - 2008-05-11 16:10:19
|
Colin Paul Adams wrote: > Can you do anything to stop these warning messages: > > gestalt_test_driver31.c:3409: warning: passing argument 1 of `GE_check_void' discards qualifiers from pointer target type > > I get a lot of them. Does it look like this bug: https://sourceforge.net/tracker/index.php?func=detail&aid=1961462&group_id=24591&atid=381937 If so, I guess I will have to add explicit type casts. -- Eric Bezault mailto:er...@go... http://www.gobosoft.com |
From: Colin P. A. <co...@co...> - 2008-05-11 16:18:07
|
>>>>> "Eric" == Eric Bezault <er...@go...> writes: Eric> Colin Paul Adams wrote: >> Can you do anything to stop these warning messages: >> >> gestalt_test_driver31.c:3409: warning: passing argument 1 of >> `GE_check_void' discards qualifiers from pointer target type >> >> I get a lot of them. Eric> Does it look like this bug: Eric> https://sourceforge.net/tracker/index.php?func=detail&aid=1961462&group_id=24591&atid=381937 Eric> If so, I guess I will have to add explicit type casts. Maybe. -- Colin Adams Preston Lancashire |
From: Colin P. A. <co...@co...> - 2008-05-16 11:39:49
|
>>>>> "Eric" == Eric Bezault <er...@go...> writes: Eric> Eric Bezault wrote: >> Colin Paul Adams wrote: >> Can you do anything to stop these warning messages: >>> >>> gestalt_test_driver31.c:3409: warning: passing argument 1 of >>> GE_check_void' discards qualifiers from pointer target type >>> >>> I get a lot of them. Eric> I added the type casts. Can you try to recompile gec and Eric> give it a try? That is a great improvement. Now I just get one warning (repeated about a dozen times) in one C file: gestalt_test_driver49.c:1920: warning: initialization from incompatible pointer type -- Colin Adams Preston Lancashire |
From: Colin P. A. <co...@co...> - 2008-05-16 14:43:17
|
>> gestalt_test_driver49.c:1920: warning: initialization from incompatible pointer type >Can you send me the C code of the corresponding function >and indicate which line is #1920? Line 1920 looks like this: {0, 1912, EIF_FALSE, &T1912f40}, This is part of a huge static array: EIF_TYPE GE_types[2384] = { The other occurrences of the warning are all of the same pattern, for instance: {0, 174, EIF_FALSE, 0}, {0, 175, EIF_FALSE, &T175f12}, there is a warning generated for the second of the two above lines, but not the first. -- Eric Bezault mailto:er...@go... http://www.gobosoft.com -- Colin Adams Preston Lancashire |
From: Eric B. <er...@go...> - 2008-05-16 15:32:40
|
Colin Paul Adams wrote: >>> gestalt_test_driver49.c:1920: warning: initialization from incompatible pointer type > >> Can you send me the C code of the corresponding function >> and indicate which line is #1920? > > Line 1920 looks like this: > > > {0, 1912, EIF_FALSE, &T1912f40}, Is T1912 an expanded type? (See its Eiffel name in the .h file.) If not, can you send me the signature of the C function T1912f40? -- Eric Bezault mailto:er...@go... http://www.gobosoft.com |
From: Eric B. <er...@go...> - 2008-05-16 16:45:23
|
Eric Bezault wrote: > Colin Paul Adams wrote: >>>> gestalt_test_driver49.c:1920: warning: initialization from >>>> incompatible pointer type >> >>> Can you send me the C code of the corresponding function >>> and indicate which line is #1920? >> >> Line 1920 looks like this: >> >> >> {0, 1912, EIF_FALSE, &T1912f40}, > > Is T1912 an expanded type? (See its Eiffel name in the .h file.) > If not, can you send me the signature of the C function T1912f40? Also, do you compile with the option "trace"? -- Eric Bezault mailto:er...@go... http://www.gobosoft.com |
From: Eric B. <er...@go...> - 2008-05-16 15:43:35
|
Eric Bezault wrote: > Eric Bezault wrote: >> Colin Paul Adams wrote: >>>>> gestalt_test_driver49.c:1920: warning: initialization from >>>>> incompatible pointer type >>> >>>> Can you send me the C code of the corresponding function >>>> and indicate which line is #1920? >>> >>> Line 1920 looks like this: >>> >>> >>> {0, 1912, EIF_FALSE, &T1912f40}, >> >> Is T1912 an expanded type? (See its Eiffel name in the .h file.) >> If not, can you send me the signature of the C function T1912f40? > > Also, do you compile with the option "trace"? I meant "exception_trace". -- Eric Bezault mailto:er...@go... http://www.gobosoft.com |
From: Colin P. A. <co...@co...> - 2008-05-16 15:54:31
|
>>>>> "Eric" == Eric Bezault <er...@go...> writes: Eric> Colin Paul Adams wrote: >>>> gestalt_test_driver49.c:1920: warning: initialization from >>>> incompatible pointer type >> >>> Can you send me the C code of the corresponding function and >>> indicate which line is #1920? >> >> Line 1920 looks like this: >> >> >> {0, 1912, EIF_FALSE, &T1912f40}, Eric> Is T1912 an expanded type? (See its Eiffel name in the .h Eric> file.) If not, can you send me the signature of the C Eric> function T1912f40? /* STDC_TEMPORARY_FILE.dispose */ extern void T1912f40(GE_call* ac, T0* C); and: <option name="trace" value="false"/> -- Colin Adams Preston Lancashire |
From: Eric B. <er...@go...> - 2008-05-12 15:32:22
|
Eric Bezault wrote: > Colin Paul Adams wrote: >> Can you do anything to stop these warning messages: >> >> gestalt_test_driver31.c:3409: warning: passing argument 1 of >> `GE_check_void' discards qualifiers from pointer target type >> >> I get a lot of them. > > Does it look like this bug: > > https://sourceforge.net/tracker/index.php?func=detail&aid=1961462&group_id=24591&atid=381937 > > > If so, I guess I will have to add explicit type casts. I added the type casts. Can you try to recompile gec and give it a try? -- Eric Bezault mailto:er...@go... http://www.gobosoft.com |
From: Eric B. <er...@go...> - 2008-05-16 12:39:43
|
Colin Paul Adams wrote: >>>>>> "Eric" == Eric Bezault <er...@go...> writes: > > Eric> Eric Bezault wrote: > >> Colin Paul Adams wrote: >>> Can you do anything to stop these warning messages: > >>> > >>> gestalt_test_driver31.c:3409: warning: passing argument 1 of > >>> GE_check_void' discards qualifiers from pointer target type > >>> > >>> I get a lot of them. > > Eric> I added the type casts. Can you try to recompile gec and > Eric> give it a try? > > That is a great improvement. > > Now I just get one warning (repeated about a dozen times) in one C file: > > gestalt_test_driver49.c:1920: warning: initialization from incompatible pointer type Can you send me the C code of the corresponding function and indicate which line is #1920? -- Eric Bezault mailto:er...@go... http://www.gobosoft.com |
From: Eric B. <er...@go...> - 2008-05-16 17:10:32
|
Eric Bezault wrote: > Eric Bezault wrote: >> Eric Bezault wrote: >>> Colin Paul Adams wrote: >>>>>> gestalt_test_driver49.c:1920: warning: initialization from >>>>>> incompatible pointer type >>>> >>>>> Can you send me the C code of the corresponding function >>>>> and indicate which line is #1920? >>>> >>>> Line 1920 looks like this: >>>> >>>> >>>> {0, 1912, EIF_FALSE, &T1912f40}, >>> >>> Is T1912 an expanded type? (See its Eiffel name in the .h file.) >>> If not, can you send me the signature of the C function T1912f40? >> >> Also, do you compile with the option "trace"? > > I meant "exception_trace". If you are indeed using the "exception_trace" option, you can recompile gec and give a try. I just committed a fix in SVN -- Eric Bezault mailto:er...@go... http://www.gobosoft.com |
From: Colin P. A. <co...@co...> - 2008-05-16 17:14:56
|
>>>>> "Eric" == Eric Bezault <er...@go...> writes: >> I meant "exception_trace". Eric> If you are indeed using the "exception_trace" option, you Eric> can recompile gec and give a try. I just committed a fix in Eric> SVN I was, so I will try again. -- Colin Adams Preston Lancashire |
From: Colin P. A. <co...@co...> - 2008-05-16 17:28:48
|
>>>>> "Eric" == Eric Bezault <er...@go...> writes: Eric> If you are indeed using the "exception_trace" option, you Eric> can recompile gec and give a try. I just committed a fix in Eric> SVN No warnings at all now. Thanks. -- Colin Adams Preston Lancashire |
From: Eric B. <er...@go...> - 2008-05-16 17:44:31
|
Colin Paul Adams wrote: > /* STDC_TEMPORARY_FILE.dispose */ > extern void T1912f40(GE_call* ac, T0* C); OK, it looks like you are in "exception_trace" mode. Please try the new version of gec in SVN. BTW, "exception_trace" in gec does not work. It will make code slower, but not more. I already pointed that out to you a few months ago when you had a similar problem with the C code generated by gec, and I thought you turned it off at that time. Apparently not. Now I wonder whether you did all your benchmarks last month with this option on as well? -- Eric Bezault mailto:er...@go... http://www.gobosoft.com |
From: Colin P. A. <co...@co...> - 2008-05-16 18:09:21
|
>>>>> "Eric" == Eric Bezault <er...@go...> writes: Eric> Colin Paul Adams wrote: >> /* STDC_TEMPORARY_FILE.dispose */ extern void T1912f40(GE_call* >> ac, T0* C); Eric> OK, it looks like you are in "exception_trace" mode. Please Eric> try the new version of gec in SVN. Eric> BTW, "exception_trace" in gec does not work. It will make Eric> code slower, but not more. I already pointed that out to you Eric> a few months ago when you had a similar problem with the C Eric> code generated by gec, and I thought you turned it off at Eric> that time. Apparently not. Now I wonder whether you did all Eric> your benchmarks last month with this option on as well? No. This is only on for the test driver for the W3C suite. It shouldn't be needed there either (it's probably a relic). -- Colin Adams Preston Lancashire |