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