Closed dwightguth closed 10 years ago
I had a look over the pull request. It looks good at a high level but I have no experience with JNI so I cannot assess that part well. Is there anybody who played with JNI before?
Regarding maven, I think the current way is ok until we figure out a group-wide solution to dependency management.
Overall, it looks good. I did not check all the details thought, so there may be small things I missed.
@cos if you are happy with my changes/comments, let me know and I will merge this.
aside from the minor issue above, it looks good to me
@andreistefanescu please review everything, not just the diff of this pull request (which excludes the initial commit)