Uploaded image for project: 'Qpid Proton'
  1. Qpid Proton
  2. PROTON-620

Remove unnecessary wrapped types from proton low level bindings

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • proton-0.8
    • None
    • None

    Description

      There are a few proton types that are exposed through the swigged low level bindings:

      • pn_atom_t (and its enclosed pn_atom_t_u)
      • pn_bytes_t
      • pn_class_t
      • pn_uuid_t
      • pn_decimal128_t
      • pn_delivery_tag_t

      pn_class_t is entirely internal to proton-c and isn't in any API - it was been removed from the 0.8 swigged bindings in a recent commit.

      The other types appear in some proton-c APIs, but the these types don't need exposing and instead there are fairly natural mappings to use in the swigged API. Indeed there are already swig typemaps which do this work for these types already.

      So these types can just be %ignored by swig when producing the bindings.

      After doing this - there are no wrapped types left in the API which I think is desirable. All proton-c structs are encapsulated in the bindings.

      Attachments

        Activity

          People

            astitcher Andrew Stitcher
            astitcher Andrew Stitcher
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: