Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor 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.

      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:
            Eric Anderson
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development