RE: [sv-champions] 5-day email vote

From: Brad Pierce <Brad.Pierce_at_.....>
Date: Mon Aug 27 2007 - 18:23:05 PDT
Neil,

I vote NO on three items -- 1609, 1556, 1723

    http://www.eda-stds.org/svdb/view.php?id=1609

        1) The new text is out of place where proposed, as becomes more
apparent if you look at the full original paragraph instead of just the
proposal in isolation.  The better place for this restriction is a
normative footnote in the BNF, specifically on the first production in
class_property of A.1.8.  It should say something like, "It shall be
illegal for a data_declaration in a class_property to be a
package_import_declaration."


    http://www.eda-stds.org/svdb/view.php?id=1556

        2)  Doesn't this break backward compatibility with traditional
Verilog by requiring an explicit lifetime keyword for all initialized
variables in static tasks and functions?

        3)  In any case, reverting this change is not necessary.  Users
are free to add 'static', and methodologies are free to require
'static', but there's no language reason to again make it mandatory for
everybody.

        4)  The example is confusing.

        5)  The comments in the example are wrong, because they say 1 2
3 3 3 3 3 3 3 3 should be printed, but actually it should be 2 3 4 4 4 4
4 4 4


    http://www.eda-stds.org/svdb/view.php?id=1723

        6) Needs an example of the new method.

        7) Not clear that that the indexing expressions are still legal
after the change from [*] to [int].
 


I vote 'yes' for all the other items, but would like to request that

    8)  Change 'Type' of 1768 in svdb from Clarification to Enhancement

    9)  The editor, when implementing 1722, change occurences of "bind
instantiation" to "bind_instantiation" and occurrences of "interface
instantiation" to "interface_instantiation".

   10)  The editor, when implementing 1768, add + and * to
cycle_delay_const_range_expression instead of adding ##[+] and ##[*] to
cycle_delay_range.

   11)  The editor, when implementing 1787, change both occurrences of
"range_value" to "value_range".

   12)  The editor, when implementing 1777, change both occurrences of
"An arbitrary number" to "Any number" and delete 'arbitrary' from "any
number of arbitrary".

   13)  The editor, when implementing 1615, change "join_noen" to
"join_none".

   14)  The editor, when implementing 1580, clarify the text to make it
read more smoothly, taking into account Shalom's earlier comments, and
reordering the penultimate sentence to "When an interface is connected
with a modport in either the module header or port connection, then, for
those types of objects legal to be listed in a modport (nets, variables,
subroutines, and clocking blocks), access through a port or through a
virtual interface is limited to only those objects listed in the
modport."

  15)  The editor, when implementing 1336, change "shall not have" to
"shall not contain".

  16)  The editor, when implmenting 888, make sure the dot after
implicit_class_handle is red.

-- Brad

-----Original Message-----
From: owner-sv-champions@eda.org [mailto:owner-sv-champions@eda.org] On
Behalf Of Neil Korpusik
Sent: Friday, August 24, 2007 5:06 PM
Subject: [sv-champions] 5-day email vote

The details are attached.
Please vote as early as possible.

Thanks,
Neil


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


-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.
Received on Mon Aug 27 18:23:25 2007

This archive was generated by hypermail 2.1.8 : Mon Aug 27 2007 - 18:23:27 PDT