Re: [Iverilog-devel] prototypes for foo_register()

[Stephen W. <st...@ic...>]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1


I admit that I do not really know what the problem is.
Sure, we should change "void sys_time_register();"
to "void sys_time_register(void);", but what else could
the compiler be warning us about?

On 07/11/2014 11:50 AM, Larry Doolittle wrote:
> Cary -
> 
> On Fri, Jul 11, 2014 at 10:42:03AM -0700, Cary R. wrote:
>> I find #4 acceptable in it's present form, but it would be best
>> to put a comment describing why the definition is there to avoid
>> confusion in the future.
> 
> Using vpi/sys_time.c as an example, we'd end up with
> 
> /* Declare a prototype first, to avoid warnings with some strict C
> compiler * options.  This prototype conforms to the VPI plugin
> standard, and is * therefore consistent with the usage in Icarus'
> sys_table.c. */ void sys_time_register(void); void
> sys_time_register(void) { s_vpi_systf_data tf_data; . .
> 
> Or is that too verbose?
> 
> I'd like to hear from Steve W. on this topic.
> 
> - Larry
> 
> ------------------------------------------------------------------------------
>
> 
_______________________________________________
> Iverilog-devel mailing list Ive...@li... 
> https://lists.sourceforge.net/lists/listinfo/iverilog-devel
> 


- -- 
Steve Williams                "The woods are lovely, dark and deep.
steve at icarus.com           But I have promises to keep,
http://www.icarus.com         and lines to code before I sleep,
http://www.picturel.com       And lines to code before I sleep."
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.19 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iEYEARECAAYFAlPAOqQACgkQrPt1Sc2b3il55ACfbLM+Eccojcp/OZYXgHSVB/qZ
CEIAmwaJv7wKZBazCezzbf9N2gLXVP/0
=Fco9
-----END PGP SIGNATURE-----