Merge lp:~paul-lucas/zorba/pjl-misc into lp:zorba
Proposed by
Paul J. Lucas
Status: | Merged |
---|---|
Approved by: | Paul J. Lucas |
Approved revision: | 11462 |
Merged at revision: | 11701 |
Proposed branch: | lp:~paul-lucas/zorba/pjl-misc |
Merge into: | lp:zorba |
Diff against target: |
63 lines (+36/-2) 4 files modified
src/runtime/csv/csv_impl.cpp (+25/-2) test/rbkt/ExpQueryResults/zorba/csv/csv-parse-address.xml.res (+3/-0) test/rbkt/Queries/zorba/csv/csv-parse-address.jq (+7/-0) test/rbkt/Queries/zorba/csv/csv-parse-address.spec (+1/-0) |
To merge this branch: | bzr merge lp:~paul-lucas/zorba/pjl-misc |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Matthias Brantner | Approve | ||
Paul J. Lucas | Approve | ||
Review via email: mp+208907@code.launchpad.net |
Commit message
Discovered a bug where strings like street addresses, e.g., "870 Market Street", will cause an exception to be thrown due to the mis-guessing about what kind of JSON token a string really is. Fixed.
Description of the change
Discovered a bug where strings like street addresses, e.g., "870 Market Street", will cause an exception to be thrown due to the mis-guessing about what kind of JSON token a string really is. Fixed.
To post a comment you must log in.
Validation queue starting for the following merge proposals: /code.launchpad .net/~paul- lucas/zorba/ pjl-misc/ +merge/ 208907
https:/
Progress dashboard at http:// jenkins. zorba.io: 8180/view/ ValidationQueue