Skip to content

dbf fields stop at char 00 #71

New issue

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

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

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

dbf fields stop at char 00 #71

wants to merge 1 commit into from

Conversation

Fil
Copy link

@Fil Fil commented Feb 8, 2022

fixes #68 #70

@Fil Fil closed this by deleting the head repository Feb 15, 2023
@mbloch
Copy link

mbloch commented Feb 15, 2023

@Fil this looks like a reasonable fix, why did you close the PR?

@Fil
Copy link
Author

Fil commented Feb 15, 2023

I didn't intend to close it, but cleaned up my personal repos and didn't realize this fork was home to a PR. Thanks for noticing!

@Fil Fil reopened this Feb 15, 2023
@mbloch
Copy link

mbloch commented Feb 16, 2023

I've seen Shapefile dbfs like this in circulation, too... hope @mbostock will consider merging.

Comment on lines +20 to +21
var v = value.subarray(i, i += f.length);
var j = v.indexOf(0);
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
var v = value.subarray(i, i += f.length);
var j = v.indexOf(0);
var v = value.subarray(i, i += f.length), j = v.indexOf(0);

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@joewoodhouse
Copy link

@mbostock any chance of releasing this? It fixes an issue I'm hitting currently

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Numeric column coming in as NULL property
4 participants