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. structure-constructor.patch
        10 kB
        Eric Anderson
      3. thrift-cpp-constructor-v2.patch
        8 kB
        Alexander Shigin
      4. thrift-cpp-constructor-v3.patch
        9 kB
        Alexander Shigin
      5. thrift-cpp-constructor-v4.patch
        9 kB
        Alexander Shigin
      6. thrift-cpp-constructor-v5.patch
        9 kB
        Alexander Shigin

        Activity

        Eric Anderson created issue -
        Eric Anderson made changes -
        Field Original Value New Value
        Attachment structure-constructor.patch [ 12385703 ]
        Eric Anderson made changes -
        Priority Major [ 3 ] Minor [ 4 ]
        Eric Anderson made changes -
        Attachment structure-constructor.patch [ 12385703 ]
        Eric Anderson made changes -
        Attachment structure-constructor.patch [ 12386502 ]
        Alexander Shigin made changes -
        Fix Version/s 0.1 [ 12313284 ]
        Alexander Shigin made changes -
        Attachment thrift-cpp-constructor-v2.patch [ 12404636 ]
        Alexander Shigin made changes -
        Attachment thrift-cpp-constructor-v3.patch [ 12404641 ]
        Alexander Shigin made changes -
        Attachment thrift-cpp-constructor-v4.patch [ 12404880 ]
        Alexander Shigin made changes -
        Attachment thrift-cpp-constructor-v5.patch [ 12404883 ]
        Bryan Duxbury made changes -
        Assignee Alexander Shigin [ shigin ]
        David Reiss made changes -
        Todd Lipcon made changes -
        Affects Version/s 0.2 [ 12313769 ]
        Fix Version/s 0.3 [ 12314451 ]
        Fix Version/s 0.1 [ 12313284 ]
        Bryan Duxbury made changes -
        Fix Version/s 0.4 [ 12314960 ]
        Fix Version/s 0.3 [ 12314451 ]
        Bryan Duxbury made changes -
        Fix Version/s 0.4 [ 12314960 ]
        Roger Meier made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Alexander Shigin [ shigin ]
        Resolution Won't Fix [ 2 ]
        Jake Farrell made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development