Skip to content
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

demonstrating error with time #6

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

gxm
Copy link

@gxm gxm commented Nov 28, 2014

I'm trying to import a gdb file which causes a NullPointerException.
I've added a test and the gdb files which demonstrates the issue.
I've tried building this in Xcode to try to fix the issue, and haven't been able to get a functional dylib.

@rdrand
Copy link

rdrand commented Dec 3, 2014

In the past when we used arcobjects I found that you often couldn't read older gdb files than the current version. If I recall correctly the unlicensed C library from ESRI that this is using was for version 10. What version of GDB file are you using?

The other consideration is what is throwing the NPE?

Building a new dylib on the Mac isn't totally trivial unfortunately. I do wish it were. If you do want to fix the issue I recommend trying to fix it on the Java side of the interface if possible.

@rdrand rdrand closed this Dec 3, 2014
@rdrand rdrand reopened this Dec 3, 2014
@gxm
Copy link
Author

gxm commented Dec 4, 2014

I am getting the gdb file from a 3rd party, I'll ask them what version of ESRI they use.

I managed to get the library to build in Xcode (yay!), and fixed the NPE. The JNI code was directly instantiating GregorianCalendar with AllocObject, which skips the ctor, which is where the TimeZone is set. I changed it to use the static Calendar.getInstance() and then set millis on that instance. You could also call a ctor on GregorianCalendar.

Alas, the fix takes me to my next issue, coordinate transforms. The lat/long in the gdb isn't in WSG84. Any chance of seeing coordinate transforms?

@gxm
Copy link
Author

gxm commented Dec 14, 2014

This Pull Request can be ignored if you accept the other - #7

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

Successfully merging this pull request may close these issues.

2 participants