Avro
  1. Avro
  2. AVRO-58 JSON Parser for C
  3. AVRO-60

JSON Parser for C needs to properly escape string text

    Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.1.0
    • Component/s: c
    • Labels:
      None
    1. AVRO-60.patch
      30 kB
      Matt Massie
    2. AVRO-60.sh
      0.1 kB
      Matt Massie

      Activity

      Matt Massie created issue -
      Matt Massie made changes -
      Field Original Value New Value
      Parent AVRO-58 [ 12428437 ]
      Issue Type Task [ 3 ] Sub-task [ 7 ]
      Hide
      Matt Massie added a comment -

      This script needs to be run to delete a few files since patch doesn't handle deletes.

      Show
      Matt Massie added a comment - This script needs to be run to delete a few files since patch doesn't handle deletes.
      Matt Massie made changes -
      Attachment AVRO-60.sh [ 12413468 ]
      Matt Massie made changes -
      Attachment AVRO-60.patch [ 12413470 ]
      Hide
      Matt Massie added a comment -

      This patch addes multi-byte character support, properly processes string escape sequences correctly, adds many more unit tests and fixes the test_json_parser unit test binary.

      The attached script AVRO-60.sh needs to be run as well.

      Show
      Matt Massie added a comment - This patch addes multi-byte character support, properly processes string escape sequences correctly, adds many more unit tests and fixes the test_json_parser unit test binary. The attached script AVRO-60 .sh needs to be run as well.
      Matt Massie made changes -
      Status Open [ 1 ] Patch Available [ 10002 ]
      Hide
      Doug Cutting added a comment -

      I just committed this. Thanks, Matt!

      Show
      Doug Cutting added a comment - I just committed this. Thanks, Matt!
      Doug Cutting made changes -
      Status Patch Available [ 10002 ] Resolved [ 5 ]
      Fix Version/s 1.0.1 [ 12314106 ]
      Resolution Fixed [ 1 ]
      Doug Cutting made changes -
      Status Resolved [ 5 ] Closed [ 6 ]
      Transition Time In Source Status Execution Times Last Executer Last Execution Date
      Open Open Patch Available Patch Available
      21d 21h 1m 1 Matt Massie 14/Jul/09 20:47
      Patch Available Patch Available Resolved Resolved
      3h 12m 1 Doug Cutting 14/Jul/09 23:59
      Resolved Resolved Closed Closed
      63d 23h 31m 1 Doug Cutting 16/Sep/09 23:31

        People

        • Assignee:
          Matt Massie
          Reporter:
          Matt Massie
        • Votes:
          0 Vote for this issue
          Watchers:
          1 Start watching this issue

          Dates

          • Created:
            Updated:
            Resolved:

            Time Tracking

            Estimated:
            Original Estimate - 1h
            1h
            Remaining:
            Remaining Estimate - 1h
            1h
            Logged:
            Time Spent - Not Specified
            Not Specified

              Development