This is not a 'complete' fix since the DBG macro is still using it; but more globally, this file should not be part of the libnfc installed files. We should rely on error codes and maybe some logging mechanism (e.g. syslog() on POSIX compliant systems and RegisterEventSource() / ReportEvent() on Windows).
- format '%ld' expects type 'long int', but argument 2 has type 'unsigned int';
- format '%ld' expects type 'long int', but argument 2 has type 'size_t'.
Tested on FreeBSD arm.
- format '%d' expects type 'int', but argument 2 has type 'long unsigned int';
- format '%d' expects type 'int', but argument 2 has type 'size_t';
- unused variable 'nti';
- unused parameter 'argc'.
1. Try to use pkg-config to locate libusb;
2. Try with libusb-config if not found;
3. Try to locate headers and lib if still not found.
This would hopefully prevent build failures in some very particular cases, such as cross compiling libnfc on a FreeBSD version for another FreeBSD version that does not use the same libusb.
http://www.freebsd.org/cgi/query-pr.cgi?pr=ports/142836
Reported by: Dmitry Marakasov
The uart_open() function may call uart_close() with a 'serial_port' structure with an invalid member 'fd' if the port was not opened successfully. The call to uart_close() was kept for consistency with the rest of the function because uart_close() is also in charge of freeing 'serial_port'.
The snprintf() function conforms to C99. On FreeBSD, sys/cdefs.h instruct us that _XOPEN_SOURCE == 500 defines _POSIX_C_SOURCE as 199506 which in turn defines __ISO_C_VISIBLE as 1990. 600 is the minimal _XOPEN_SOURCE value we can use to allow us using C99 functions.
Reported by: tinderbox
When I introduced headers separation (aka includes-revamp branch), the libnfc source code includes have been changed from '#include "../xx/nfc*.h"' to '#include <nfc/nfc*>' (when compiling, the source include dir (src/include) has precedence over '${prefix}/include'). For the sake of consistency, update libnfc.pc includedir so that compiling third party code using '#include <nfc/nfc*>' just works (tm). This provides much more consistent headers since they are usually referred relative to the includedir (exceptions are e.g. glib, gtk, xorg... quite bigger than libnfc I could say).
Since March 9, 2009; the FreeBSD operating system base ships with libusb. As a consequence, the devel/libusb ports that used to provide userland programs access to USB devices is useless and so marked as IGNORE for recent versions of FreeBSD. This patch detects the FreeBSD version configure is run on and conditionally ignore the libusb detection if relevant.
Obtained from: Ooo vcs.
The various SCardConnect() can block if a device is already opened in PC/SC potentially leading to deadlocks. Avoiding using this and matching the device name against a list of supported devices instead.
- Add driver agnostic nfc_pick_device(), nfc_list_devices();
- New API function: nfc_list_devices();
- PCSC Context sharing for acr122 driver;
- List all devices in nfc-list(1);
- Various code fixes and cleanup;
- Remove warnings when compiling;
- Merge r191:199 from trunk \_°< Coin!
- acr122: add acr122_pick_device(), acr122_list_devices();
- Add device name to device descriptions (nfc_device_desc_t) to select a given PCSC device;
- Use constants for driver names;
- Various code cleanups and enhancements.
Romain Tartiere