CORBA 2.4 NO IDEA Avatar
  1. OMG Issue

CORBA24 — Non-escaped keywords in published IDL

  • Key: CORBA24-95
  • Legacy Issue Number: 3685
  • Status: closed  
  • Source: Triodia Technologies Pty Ltd ( Michi Henning)
  • Summary:

    I know that this is probably not the best RTF to send this to, but the
    issue spans RTFs and we don't have RTFs for the collection or the life cycle
    service, so I'm sending this to the Core RTF for want of a better task force...

    In the CosLifeCycle IDL (formal/98-10-01.idl), we have:

    module CosLifeCycle{

    typedef CosNaming::Name Key;
    typedef Object Factory;
    typedef sequence <Factory> Factories;

    The two occurences of "Factory" are illegal. According to the comment
    at the beginning of the module, this should read:

    module CosLifeCycle{

    typedef CosNaming::Name Key;
    #ifdef NO_ESCAPED_IDENTIFIERS
    typedef Object Factory;
    typedef sequence <Factory> Factories;
    #else
    typedef Object _Factory;
    typedef sequence <_Factory> Factories;
    #endif

    The same problem also appears in formal/98-10-15.idl.

    Also in formal/98-10-01.idl:

    // C o l l e c t i o n F a c t o r i e s
    interface CollectionFactories : CollectionFactory {
    boolean add_factory (
    in Istring collection_interface,
    in Istring impl_category,
    in Istring impl_interface,
    in CollectionFactory factory);

    // ...

    // R A C o l l e c t i o n F a c t o r i e s
    interface RACollectionFactories : RACollectionFactory {
    boolean add_factory (
    in Istring collection_interface,
    in Istring impl_category,
    in Istring impl_interface,
    in RACollectionFactory factory);

    Both operation definitions need the same #ifdef to map away from the
    "factory" keyword that is used as a parameter name.

    That same problem also appears in formal/98-10-03.idl.

    I guess the IDL in the formal CORBAservices documents should be fixed too.

  • Reported: CORBA 2.3.1 — Thu, 6 Jul 2000 04:00 GMT
  • Disposition: Resolved — CORBA 2.4
  • Disposition Summary:

    Fix IDL as suggested

  • Updated: Fri, 6 Mar 2015 20:58 GMT