Tuscany
  1. Tuscany
  2. TUSCANY-1368

C++ SDO portability: class interface Type off-spec enum values

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: Cpp-M3
    • Fix Version/s: None
    • Component/s: C++ SDO, C++ Specification
    • Labels:
      None

      Description

      C++ SDO specification class interface Type has enum values ("OpenDataObjectType", "num_type")which are not in the specification, and are being used externally by SCA. It would seem that for SDO portability these should be taken internal to SDO or submitted to the spec committee.

      ----Original Message----
      From: Pete Robbins robbinspg@googlemail.com
      Sent: Thursday, June 21, 2007 9:21 AM
      To: tuscany-dev@ws.apache.org
      Subject: Re: SDO spec compliance/portability: Type enums

      the num_type is just a convenient way to know the extent of an enum and is common practice. I guess the OpenDataObjectType must be a Tuscany specific extension to handle open types. I'd need to do some research to determine if this is missing from the spec or can be removed.

      Please raise a Jira for the renaming of IntegerType.

      It may be useful to raise Jiras for all these issues so we can track them.

      Cheers,

      On 21/06/07, Michael Yoder <yoder@roguewave.com> wrote:
      >
      > Hi,
      >
      > The Tuscany SDO C++ class Type::Types enum has these values which do
      > not appear in the C++ 2.1 specification:
      >
      > - OpenDataObjectType
      > - num_type
      >
      > Would it be appropriate to file a bug to remove the additional values?
      > Or alternatively a bug for them to be
      > submitted to the spec committee?
      >
      > In addition the 2.1 spec renamed the enum value "IntegerType" to
      > "IntType".
      >
      > Would it be appropriate to file a bug to have this value renamed?
      >
      > Thanks,
      >
      > Michael Yoder
      > Software Developer
      > Rogue Wave Software
      >
      > ---------------------------------------------------------------------
      > To unsubscribe, e-mail: tuscany-dev-unsubscribe@ws.apache.org
      > For additional commands, e-mail: tuscany-dev-help@ws.apache.org
      >
      >


      Pete

      1. TUSCANY-1368.txt
        227 kB
        Michael Yoder

        Activity

        Hide
        Pete Robbins added a comment -

        adding specification component

        Show
        Pete Robbins added a comment - adding specification component
        Hide
        Michael Yoder added a comment -

        This patch updates the Type.h interface to be spec compliant. Enum values and public members not in the spec are removed from the interface. Also related/ripple 2.1 spec changes:

        • commonj.sdo#Int type added to factory (maps to enum Types::IntType)
        • commonj.sdo#BigInteger type rennamed commonj.sdo#Integer (maps to enum Types::BigInteger)
        • commonj.sdo#BigDecimal type renamed commonj.sdo#Decimal (maps to enum Types::BigDecimal)
        • set/getInteger accessors renamed to set/getInt (map to commonj.sdo#Int and xsd:int)
        Show
        Michael Yoder added a comment - This patch updates the Type.h interface to be spec compliant. Enum values and public members not in the spec are removed from the interface. Also related/ripple 2.1 spec changes: commonj.sdo#Int type added to factory (maps to enum Types::IntType) commonj.sdo#BigInteger type rennamed commonj.sdo#Integer (maps to enum Types::BigInteger) commonj.sdo#BigDecimal type renamed commonj.sdo#Decimal (maps to enum Types::BigDecimal) set/getInteger accessors renamed to set/getInt (map to commonj.sdo#Int and xsd:int)
        Hide
        Michael Yoder added a comment -

        Resolved with applied patch.

        Show
        Michael Yoder added a comment - Resolved with applied patch.

          People

          • Assignee:
            Unassigned
            Reporter:
            Michael Yoder
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development