Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Won't Fix
    • Affects Version/s: 0.2
    • Fix Version/s: None
    • Component/s: C++ - Compiler
    • Labels:
      None
    • Environment:

      Tested on debian stable (etch)

    • Patch Info:
      Patch Available

      Description

      We're mostly using the C++ interface to thrift, and I wanted to be
      able to initialize structures more easily than a whole series of lines
      that set each of the parameters separately. Attached is a patch that
      adds a constructor to the C++ objects that allow you to fully
      initialize an object in a single call. I added two tests for this,
      one in the DebugProtoTest.cpp file, and one in the
      OptionalRequiredTest.cpp. The patch is against the 20080411p1 release.

        Attachments

        1. ASF.LICENSE.NOT.GRANTED--0001-THRIFT-72.-cpp-Add-a-struct-constructor-that-takes.patch
          9 kB
          David Reiss
        2. thrift-cpp-constructor-v5.patch
          9 kB
          Alexander Shigin
        3. thrift-cpp-constructor-v4.patch
          9 kB
          Alexander Shigin
        4. thrift-cpp-constructor-v3.patch
          9 kB
          Alexander Shigin
        5. thrift-cpp-constructor-v2.patch
          8 kB
          Alexander Shigin
        6. structure-constructor.patch
          10 kB
          Eric Anderson

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              anderse Eric Anderson
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: