|
From: Larry D. <ldo...@re...> - 2014-07-11 18:51:01
|
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
|