Skip to content

BLD: fix inf processing in read_csv #4373

New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
jreback opened this issue Jul 26, 2013 · 5 comments
Closed

BLD: fix inf processing in read_csv #4373

jreback opened this issue Jul 26, 2013 · 5 comments
Labels
Build Library building on various platforms
Milestone

Comments

@jreback
Copy link
Contributor

jreback commented Jul 26, 2013

as @dieterv77 points out, fix for this in #4220 of issue #4219

does not build on windows

cc @jtratner

can change to using a HashTable (I think we discussed this anyhow)

?

@jtratner
Copy link
Contributor

Yes. Yeehaw, time for some hash tables...
On Jul 26, 2013 2:47 PM, "jreback" [email protected] wrote:

as @dieterv77 https://github.com/dieterv77 points out, fix for this in
#4220 #4220 of issue #4219#4219

does not build on windows

cc @jtratner https://github.com/jtratner

can change to using a HashTable (I think we discussed this anyhow)

?


Reply to this email directly or view it on GitHubhttps://github.com//issues/4373
.

@jtratner
Copy link
Contributor

I think this was fixed already - I thought I saw a commit that added a compat.c and used a define to fix it.

@jreback
Copy link
Contributor Author

jreback commented Jul 31, 2013

really??? I pushed your inf change but I didn't see anything else

can u find the ref?

@jtratner
Copy link
Contributor

@jreback stealth commit by @dieterv77 - 7a15bdd

@jreback
Copy link
Contributor Author

jreback commented Jul 31, 2013

perfect!

@jreback jreback closed this as completed Jul 31, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build Library building on various platforms
Projects
None yet
Development

No branches or pull requests

2 participants