Issue 3226: PSS: factory_dcl syntax                                                                                                                                                                                 (pss-ftf)
Source: Humboldt-Universitaet (Mr. Martin von Loewis, loewis@informatik.hu-berlin.de)
Nature: Uncategorized Issue
Severity: 
Summary: The draft PSS specification, in section 1.3.2, uses the production
<factory_dcl>. This production is not defined in the grammar
summary. Instead, it is later given as

<factory_dcl> ::= "factory" <identifier> <factory_parameters> 
<factory_parameters> ::= 
   "(" <simple_declarator> [{ "," <simple_declarator> }*] ")" 
   |"(" ")"

This definition contradicts with the production <factory_dcl> from the
CORBA components draft (ptc/99-10-04), where it is defined as

*lt;factory_dcl> ::= 
   "factory" <identifier> "(" [ <init_param_decls> ] ")" [
   <raises_expr> ]

Here, the init_param_decls where introduced together with OBV
(formal/98-12-01), where each parameter has an "in" direction and a
type, and a name.

There is also potential confusion with the init_dcl, which represents
factories for value types.

At a minimum, one of the production names should be changed, and it
should be made clear which syntax is used for the "factory" keyword in
what context. It would be even better if the syntax for factories is
always the same.
Resolution: accepted
Revised Text: (a) in the PSS spec, rename the production factory_dcl to psdl_factory_decl
and
(b) add the psdl_factory_decl production to the grammar summary.
The fundamental difference between PSDL factories and valuetype factories is that the
implementation of PSDL factories is generated by the PSDL compiler, whereas the
implementation of IDL value factories is provided by the CORBA developer. This is why
PSDL factory operations cannot accept arbitrary parameters.
Revised Text:
See http://cgi.omg.org/pub/pss_ftf/proposals.pdf (color MAGENTA).
Actions taken:
January 17, 2000: received issue
October 3, 2001: closed issue
Discussion: 

End of Annotations:=====
Date: Mon, 17 Jan 2000 18:04:19 +0100
Message-Id: <200001171704.SAA16982@pandora>
From: Martin von Loewis <loewis@informatik.hu-berlin.de>
To: issues@omg.org
Subject: PSS: factory_dcl syntax
User-Agent: SEMI/1.13.3 (Komaiko) FLIM/1.12.5 (Hirahata) Emacs/20.4
(sparc-sun-solaris2.5.1) MULE/4.0 (HANANOEN)
MIME-Version: 1.0 (generated by SEMI 1.13.3 - "Komaiko")
Content-Type: text/plain; charset=US-ASCII
X-UIDL: R/md9??id9cQgd9n+Q!!

The draft PSS specification, in section 1.3.2, uses the production
<factory_dcl>. This production is not defined in the grammar
summary. Instead, it is later given as

<factory_dcl> ::= "factory" <identifier> <factory_parameters> 
<factory_parameters> ::= 
   "(" <simple_declarator> [{ "," <simple_declarator> }*] ")" 
   |"(" ")"

This definition contradicts with the production <factory_dcl> from the
CORBA components draft (ptc/99-10-04), where it is defined as

<factory_dcl> ::= 
   "factory" <identifier> "(" [ <init_param_decls> ] ")" [
   <raises_expr> ]

Here, the init_param_decls where introduced together with OBV
(formal/98-12-01), where each parameter has an "in" direction and a
type, and a name.

There is also potential confusion with the init_dcl, which represents
factories for value types.

At a minimum, one of the production names should be changed, and it
should be made clear which syntax is used for the "factory" keyword in
what context. It would be even better if the syntax for factories is
always the same.

Regards,
Martin