Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fixed a problem with processing CSV lines containing empty fields. #1

Merged
merged 1 commit into from May 4, 2013

Conversation

Projects
None yet
2 participants

Example:

  • Before the fix:

    (CSV:parse-string "1,2,3,,4,,5,,")
    (("1" "2" "3"))

  • After the fix:

    (CSV:parse-string "1,2,3,,4,,5,,")
    (("1" "2" "3" "" "4" "" "5" ""))

Discussion at the newLISP Fan Club about this issue is at http://newlispfanclub.alh.net/forum/viewtopic.php?f=16&t=4334.

Thanks! —Rick (rickyboy)

Rick Hanson Fixed a problem with processing CSV lines containing empty fields.
Example:
- Before the fix:
  > (CSV:parse-string "1,2,3,,4,,5,,")
  (("1" "2" "3"))
- After the fix:
  > (CSV:parse-string "1,2,3,,4,,5,,")
  (("1" "2" "3" "" "4" "" "5" ""))

Discussion at the newLISP Fan Club about this issue is at
http://newlispfanclub.alh.net/forum/viewtopic.php?f=16&t=4334
959a624

@kanendosei kanendosei pushed a commit that referenced this pull request May 4, 2013

Kanen Flowers Merge pull request #1 from cryptorick/empty-field-fix
Fixed a problem with processing CSV lines containing empty fields.
6af47d6

@kanendosei kanendosei merged commit 6af47d6 into kanendosei:master May 4, 2013

Owner

kanendosei commented May 4, 2013

Merged

On May 3, 2013, at 1:08 PM, Rick Hanson notifications@github.com wrote:

Example:

Before the fix:

(CSV:parse-string "1,2,3,,4,,5,,")
(("1" "2" "3"))

After the fix:

(CSV:parse-string "1,2,3,,4,,5,,")
(("1" "2" "3" "" "4" "" "5" ""))

Discussion at the newLISP Fan Club about this issue is at http://newlispfanclub.alh.net/forum/viewtopic.php?f=16&t=4334.

Thanks! —Rick (rickyboy)

You can merge this Pull Request by running

git pull https://github.com/cryptorick/artful-newlisp empty-field-fix
Or view, comment on, or merge it at:

#1

Commit Summary

Fixed a problem with processing CSV lines containing empty fields.
File Changes

M csv.lsp (4)
Patch Links:

https://github.com/kanendosei/artful-newlisp/pull/1.patch
https://github.com/kanendosei/artful-newlisp/pull/1.diff

@cryptorick cryptorick deleted the unknown repository branch May 4, 2013

Thanks, Kanen. Hope everything is going well for you. Cheers! --Rick

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment