Thrift
  1. Thrift
  2. THRIFT-1405

Delphi compiler does not generates container serializer properly.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.8
    • Fix Version/s: 0.8
    • Component/s: Delphi - Compiler
    • Labels:

      Description

      There is a bug in the function that generates serializer of containers.

        Activity

        Hide
        Hudson added a comment -

        Integrated in Thrift #308 (See https://builds.apache.org/job/Thrift/308/)
        THRIFT-1405: Delphi compiler does not generates container serializer properly
        Client: delphi
        Patch: Kenjiro Fukumitsu

        Fixess a bug in the function that generates serializer of containers

        jfarrell : http://svn.apache.org/viewvc/?view=rev&rev=1189009
        Files :

        • /thrift/trunk/compiler/cpp/src/generate/t_delphi_generator.cc
        Show
        Hudson added a comment - Integrated in Thrift #308 (See https://builds.apache.org/job/Thrift/308/ ) THRIFT-1405 : Delphi compiler does not generates container serializer properly Client: delphi Patch: Kenjiro Fukumitsu Fixess a bug in the function that generates serializer of containers jfarrell : http://svn.apache.org/viewvc/?view=rev&rev=1189009 Files : /thrift/trunk/compiler/cpp/src/generate/t_delphi_generator.cc
        Hide
        Jake Farrell added a comment -

        Committed, thanks

        Show
        Jake Farrell added a comment - Committed, thanks
        Hide
        Jens Geyer added a comment -

        Problem fixed, all test cases passed: +1 from me

        Show
        Jens Geyer added a comment - Problem fixed, all test cases passed: +1 from me
        Hide
        Kenjiro Fukumitsu added a comment -

        Patch to fix this problem.

        • Moved oprot.WriteListEnd() outside of element's loop.
        Show
        Kenjiro Fukumitsu added a comment - Patch to fix this problem. Moved oprot.WriteListEnd() outside of element's loop.

          People

          • Assignee:
            Kenjiro Fukumitsu
            Reporter:
            Kenjiro Fukumitsu
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development