dain / leveldb

Port of LevelDB to Java
Apache License 2.0
1.54k stars 429 forks source link

deal with null here - happens e.g. when platform is android #52

Closed ligi closed 8 years ago

ligi commented 9 years ago

there was a workaround described by @romanman for #43 and the issue was closed. But we can avoid the crash in the first place by dealing with the null-value that is possible here

chakrit commented 9 years ago

Can we some more android love here and have this merged?

cvermilion commented 9 years ago

It looks like sun.arch.data.model isn't portable but I think os.arch should be -- maybe that would be a better way to test for 32 vs 64 bit?

dain commented 8 years ago

Merged, thanks!