Hacker Newsnew | past | comments | ask | show | jobs | submitlogin

Thanks for the correction regarding escaping, but I think you went a little overboard on the other alterations:

> Omitted optional quotes for fields that don't need them

I think it's good policy to always wrap your contents in quotes regardless of whether you have a delimiter that needs quoting. And in fact many CSV marshallers will do just this.

> And there is no space after a comma

That was added purely for readability on HN. I agree it's not how you'd normally marshal the contents.



Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: