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

A few minor Java wrapper changes #398

Merged
merged 5 commits into from
Sep 1, 2014
Merged

Conversation

nneonneo
Copy link
Contributor

@nneonneo nneonneo commented Jul 3, 2014

Here's a few minor Java changes, for happier development.

  • Implement every API in fakenect, which resolves linking errors that can occur when using Java with fakenect.
  • Bump the version number of the JAR
  • Add a Makefile for the JAR (mostly since I never remember the right mvn command to use)
  • Build a JAR with dependencies by default, which makes it slightly less painful to deploy
  • Like the other wrappers, don't set a default depth or video mode (the default modes that libfreenect sets are fine).
  • Order the data buffers correctly (according to native byte order instead of the default BIG_ENDIAN), so that you can make ShortBuffers from them that work properly

nneonneo added 5 commits July 2, 2014 20:42
Signed-off-by: Robert Xiao <brx@cs.cmu.edu>
Signed-off-by: Robert Xiao <brx@cs.cmu.edu>
Signed-off-by: Robert Xiao <brx@cs.cmu.edu>
…Java wrapper

Signed-off-by: Robert Xiao <brx@cs.cmu.edu>
The defaults from libfreenect are fine.

Signed-off-by: Robert Xiao <brx@cs.cmu.edu>
@piedar piedar added this to the v0.5.0 milestone Jul 3, 2014
@piedar piedar modified the milestones: v0.5.1, v0.5.0 Jul 15, 2014
@piedar piedar force-pushed the master branch 2 times, most recently from e241863 to 4e489af Compare September 1, 2014 21:07
piedar added a commit that referenced this pull request Sep 1, 2014
A few minor Java wrapper changes

Reviewed-by: Benn Snyder <benn.snyder@gmail.com>
@piedar piedar merged commit 2994c6d into OpenKinect:master Sep 1, 2014
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