RE: [sv-cc] RE: Draft 7 erratum in t_vpi_arrayvalue struct?

Abi-
Apologies for just now getting to this!
 
I agree the "**times" needs correcting, and YES there is an extra brace
at the top of the arrayval typedef !  Good catches.
 
Re-tracing my thinking on "**times", I may have intended to have this
refer to an array of pointers to time structs, but this I agree does not
fit into the user-allocated value storage vs. VPI-allocated value
storage paradigm, i.e. it's unnecessarily complex and should be
"*times".
 
[ Charlie]
Is there any chance we (SV-CC) could include/approve this correction to
Annex L, which I think we are reviewing re. 2226 anyway ?
 
The changes would be:
 
    1)   typedef struct t_vpi_arrayvalue { 

          {

          -- to --

          typedef struct t_vpi_arrayvalue          // Left-brace removed
here

          {

 

    2)   struct t_vpi_time **times; /* time values */

          -- to --
 
          struct t_vpi_time *times; /* time values */
 
Regards,
Chuck
 



________________________________

	From: owner-sv-cc@eda.org [mailto:owner-sv-cc@eda.org] On Behalf
Of Moorhouse, Abigail
	Sent: Tuesday, September 16, 2008 2:17 PM
	To: Moorhouse, Abigail; SV-CC
	Subject: [sv-cc] RE: Draft 7 erratum in t_vpi_arrayvalue struct?
	
	
	... I think there is an extra left brace in there too.

________________________________

	From: Moorhouse, Abigail 
	Sent: Tuesday, September 16, 2008 11:14 AM
	To: SV-CC
	Subject: Draft 7 erratum in t_vpi_arrayvalue struct?
	
	
	hi all
	In the new t_vpi_arrayvalue struct I believe the 'times' element
of the union is mis-defined as it has double-indirection (two ** rather
than one *).
	 
	Chuck, you probably should double-check this, I might be missing
something.
	Abi

	-- 
	This message has been scanned for viruses and 
	dangerous content by MailScanner <http://www.mailscanner.info/>
, and is 
	believed to be clean. 


-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.