Remove C99 syntax from fakenect.c #369
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Recent commit a05075a uses C99 syntax on line 101 of fakenect.c: for (int c = /* snip */ However, the build system compiles (at least on my machine) using C89, causing the build to fail. Moving the declaration of int c to the top of the function solves the problem.
yarikoptic
added a commit
to yarikoptic/libfreenect
that referenced
this pull request
Feb 14, 2015
libfreenect v0.4.0 Oracle * tag 'v0.4.0': Update for v0.4.0 All headers (including libfreenect.hpp) are installed to `$PREFIX/include/libfreenect/`. All libfreenect files have `#include "libfreenect.h"` rather than `#include <libfreenect.h>`. This works during build and after installation all include files are in the same directory. Rename libfreenect-audio.h -> libfreenect_audio.h and libfreenect-registration.h -> libfreenect_registration.h. This makes them consistent with other libraries. wrappers/cpp: Call freenect_process_events_timeout in loop to prevent freezes - fixes OpenKinect#223 OpenNI2-FreenectDriver: Fix silly silly string bugs; improve logging Fix ebuild for filename change OpenNI2-FreenectDriver: Convert all printf and cout statements to log messages Invoke fwfetcher.py with python2 - fixes OpenKinect#370 Refine README.md Remove C99 syntax - fixes OpenKinect#369 Shiny new README.md Beautify ebuild and add audio USE flag Read int from fgetc for EOF comparison - fixes OpenKinect#365 Replaced missing variable - fixes OpenKinect#366 Post-merge cleanup small changes to allow compile on windows / visual studio Removed memset() after free() causing sporadic crashes when unplugging devices Fix ebuild Supports loading audio fw from memory as well as disk. Supports tilt,accel and led for K4W and 1473 device. Adds keep alive for K4W and 1473. Signed-off-by: Theodore Watson theo@openframeworks.cc (ofTheo) This code sets the LED of a 1473 Kinect or K4W Kinect to red on startup. This seems to fix a known issue for OS X users where the device renumerates within a small time period and causes a freeze. Closes OpenKinect#340 and addresses OpenKinect#316. Signed-off-by: Theodore Watson <theo@openframeworks.cc> (ofTheo) Remove outdated ebuild Add manpages for fakenect and fakenect-record Prefix freenect- to the name of the installed demos Rename record binary of fakenect as fakenect-record
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Recent commit a05075a uses C99 syntax
on line 101 of fakenect.c:
However, the build system compiles (at least on my machine) using C89,
causing the build to fail. Moving the declaration of int c to the top
of the function solves the problem.