RE: [sv-ec] E-mail Ballot Due Wenesday, June 10, 8AM PDT

From: Bresticker, Shalom <shalom.bresticker_at_.....>
Date: Wed Jun 03 2009 - 04:06:27 PDT
If I had a vote, I would vote NO on Mantis 2575 for the reason detailed in a bug note:

The proposal makes the following BNF change:

FROM
 
constant_primary ::=
...
| ps_parameter_identifier constant_select

TO

constant_primary ::=
...
| [package_scope | class_scope] parameter_identifier constant_select

That is, "ps_parameter_identifier" is replaced with "[package_scope | class_scope]".

I see two problems with this:

A. 

ps_parameter_identifier ::=
  [ package_scope ] parameter_identifier
| { generate_block_identifier [ [ constant_expression ] ] . } parameter_identifier

Making the change in constant_primary will remove the second alternative of ps_parameter_identifier (a generate block parameter) from constant_primary.

B. ps_parameter_identifier appears in two other places in the BNF:

simple_type ::= integer_type | non_integer_type | ps_type_identifier | ps_parameter_identifier

assignment_pattern_expression_type ::=
  ps_type_identifier
| ps_parameter_identifier
| integer_atom_type
| type_reference

Does it not make sense to make this change (to add class_scope) in those places as well?

It seems to me more logical to make the change in ps_parameter_identifier:

ps_parameter_identifier ::=
  [ package_scope | class_scope ] parameter_identifier
| { generate_block_identifier [ [ constant_expression ] ] . } parameter_identifier


I still have not gotten any feedback on this comment.

Shalom
---------------------------------------------------------------------
Intel Israel (74) Limited

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.


-- 
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.
Received on Wed Jun 3 04:07:09 2009

This archive was generated by hypermail 2.1.8 : Wed Jun 03 2009 - 04:07:18 PDT