Bastien Nocera
1a376c1bfa
lib: Remove 2 more functions from the drivers API
2018-05-29 13:34:18 +02:00
Bastien Nocera
ef807d9d0e
lib: s/array_n_elements/G_N_ELEMENTS/
2018-05-29 13:34:18 +02:00
Bastien Nocera
2a4893d946
lib: fp_imgdev_*() functions are only used in drivers
2018-05-29 13:34:18 +02:00
Bastien Nocera
cee061b363
lib: fpi_ssm_start_subsm() is only used in drivers
2018-05-29 13:34:18 +02:00
Bastien Nocera
1bbdf304ab
lib: Remove never defined fpi_ssm_has_completed()
2018-05-29 13:34:18 +02:00
Bastien Nocera
6155068f9e
lib: Split off some fp_img related functions
...
Some are internal, some are for drivers.
2018-05-29 13:34:18 +02:00
Bastien Nocera
0c4e3bb1c4
lib: Remove drivers definitions from drivers API
2018-05-29 13:34:18 +02:00
Bastien Nocera
4547ff0c19
lib: Remove a number of data types from drivers API
...
They're unused in drivers.
2018-05-29 13:34:18 +02:00
Bastien Nocera
b7cce4d91d
lib: Make fp_minutiae opaque
...
And fp_minutia private.
2018-05-29 13:34:18 +02:00
Bastien Nocera
475250ce71
lib: Split off some polling related functions
2018-05-29 13:34:18 +02:00
Bastien Nocera
f40f231a63
lib: Make fp_img_dev structure opaque
2018-05-29 13:34:18 +02:00
Bastien Nocera
d83d92adf2
lib: Make fp_dev structure opaque
2018-05-29 13:34:18 +02:00
Bastien Nocera
d15282bff1
lib: Remove minutiae utils from internal header
...
It's only used by drivers.
2018-05-29 13:34:18 +02:00
Bastien Nocera
00637c4f0b
lib: Remove fprint_get_drivers() from drivers API
...
Drivers don't need to get a list of drivers.
2018-05-29 13:34:18 +02:00
Bastien Nocera
2e035a7f45
lib: Make fpi_ssm opaque for drivers
...
We shouldn't access fpi_ssm struct fields directly in drivers, so add
accessor and setter functions for the common uses.
2018-05-29 13:34:18 +02:00
Bastien Nocera
17577175f8
lib: Remove fpi_driver_to_img_driver() from drivers API
...
It's only used internally to the library.
2018-05-29 13:34:18 +02:00
Bastien Nocera
933fab6ed4
lib: Remove array_n_elements() from internal header
...
It's unused.
2018-05-29 13:34:18 +02:00
Bastien Nocera
7e5661caff
lib: Don't include assembling.h in internal header
...
It's not needed.
2018-05-29 13:34:18 +02:00
Bastien Nocera
19dfb138a6
drivers: Use new drivers_api.h in drivers
2018-05-29 13:34:18 +02:00
Bastien Nocera
0930f2614c
lib: Use public header in helper files
...
aeslib.h and assembling.h don't need access to internal data structures,
so make them include the public fprint.h rather than the internal header
fp_internal.h.
2018-05-29 13:34:18 +02:00
Bastien Nocera
1b20521e5c
lib: Add drivers_api.h file
...
This will hide library internals, for the purpose of making the drivers
API documentable.
2018-05-29 13:34:18 +02:00
Bastien Nocera
0e351db91a
lib: Fix new BUG_ON() implementation again
...
This time, the macro didn't wrap the condition, resulting in calls
like BUG_ON(size > 100) being expanded to g_assert(!size > 100), when
what we wanted was BUG_ON(!(size > 100)).
See 9cca501650
,
ff09456cf5
, and egg on my face.
2018-05-25 17:38:40 +02:00
Bastien Nocera
9cca501650
lib: Fix new BUG_ON() implementation
...
A thinko reversed the meaning of the BUG_ON() condition. This fixes
the inverted implementation in commit
ff09456cf5
.
2018-05-25 16:12:29 +02:00
Bastien Nocera
2481cbe4ab
poll: Add some details about how mainloop integration works
2018-05-25 15:58:29 +02:00
Bastien Nocera
bc3959d1e0
lib: Fix internal deprecation warning
...
Remove deprecation warning that comes from an internal function, it
serves no purpose, and the function itself is deprecated.
Follow-up from commit 1f0079a274
.
2018-05-25 13:45:13 +02:00
Bastien Nocera
1ac815e457
examples: Fix integer overflow warnings
2018-05-25 13:40:33 +02:00
Bastien Nocera
55b83062d0
img: Fix fd leaks
...
Spotted by maddin200@aol.com
https://bugs.freedesktop.org/show_bug.cgi?id=90197
2018-05-25 13:30:35 +02:00
Mark Harfouche
391f77ce5e
aes1610: Fix compilation warning in aes1610
...
Fixes:
drivers/aes1610.c:736:34: warning: ‘stop_reader’ defined but not used [-Wunused-const-variable=]
https://bugs.freedesktop.org/show_bug.cgi?id=105429
2018-05-25 13:21:17 +02:00
Bastien Nocera
bc30a3d2e5
aes2501: Fix state machine never using "init_3" state
...
This fixes this warning by the same token:
drivers/aes2501.c:671:34: warning: ‘init_3’ defined but not used [-Wunused-const-variable=]
https://bugs.freedesktop.org/show_bug.cgi?id=105429
2018-05-25 13:21:17 +02:00
Bastien Nocera
ff5de4ff03
lib: Replace open-coded array_n_elements()
...
Replace it with glib's version, which already exists.
https://bugs.freedesktop.org/show_bug.cgi?id=106280
2018-05-25 13:20:34 +02:00
Bastien Nocera
f433a4d67c
upekts: Fix compilation warning
...
libfprint/drivers/upekts.c: In function ‘alloc_send_cmd_transfer’:
libfprint/drivers/upekts.c:161:2: warning: ‘strncpy’ output truncated before terminating nul copying 4 bytes from a string of the same length [-Wstringop-truncation]
strncpy(buf, "Ciao", 4);
^~~~~~~~~~~~~~~~~~~~~~~
Replace with memcpy() to only copy the 4 bytes we need.
https://bugs.freedesktop.org/show_bug.cgi?id=106281
2018-05-25 13:19:31 +02:00
Bastien Nocera
994061af44
drivers: Simplify headers
...
Remove all the headers already included through "fp_internal.h" such as
<libusb.h> and <errno.h>, include "assembling.h" and "driver_ids.h" there
as well to avoid doing it in (almost) every driver.
2018-05-24 17:02:15 +02:00
Bastien Nocera
501020921e
vfs301_proto: Use GLib helpers
...
No need to roll out own min() implementation, or use the bare assert().
2018-05-24 17:00:17 +02:00
Bastien Nocera
53c09405b2
etes603: Use GLib's g_assert()
...
Instead of requiring assert.h to be included.
2018-05-24 16:57:14 +02:00
Bastien Nocera
1f0079a274
data: Mark all the fp_dscv_print functions as deprecated
...
As this is pretty much copy/pasted in fprintd, and should instead be
implemented by whatever system actually stores the data, rather than
in a generic but not quite "fits-all-purpose" way.
https://bugs.freedesktop.org/show_bug.cgi?id=106550
2018-05-24 13:21:40 +02:00
Bastien Nocera
15afe43cf0
examples: Replace deprecated fp_set_debug()
2018-05-24 12:43:53 +02:00
Bastien Nocera
eb8f7ba3b2
lib: Mark fp_set_debug() as deprecated
...
As per the documentation.
2018-05-24 12:42:18 +02:00
Bastien Nocera
901a6f7fed
lib: Add macro for deprecated functions
2018-05-24 12:41:35 +02:00
Bastien Nocera
fc92f62136
build: Remove the need to modify sources for new drivers
...
Instead of having to modify both fp_internal.h to list each driver
definition structure, and core.c to add those drivers to arrays we
can loop over, generate both of those using meson.
2018-05-24 12:23:39 +02:00
Bastien Nocera
ff09456cf5
lib: Use g_assert* to implement BUG() and BUG_ON() assertions
2018-05-23 19:14:33 +02:00
Bastien Nocera
32fcfde86b
lib: Use GLib and libusb directly for debug output
...
Use GLib internally to output debug information, and tell about
libusb's LIBUSB_DEBUG envvar for libusb debug.
fp_set_debug() is now a no-op.
https://bugs.freedesktop.org/show_bug.cgi?id=106552
2018-05-23 19:13:08 +02:00
Bastien Nocera
363a1b3371
lib: Replace empty fp_dbg() calls
...
GLib won't like them, so use G_DEBUG_HERE() instead.
2018-05-23 19:11:07 +02:00
Bastien Nocera
c376c6fb02
lib: Fix type mismatch warnings in debug output
2018-05-23 19:11:05 +02:00
Bastien Nocera
8e6e23b8d0
build: Always enable debugging logging
...
We shouldn't need to specifically enable debug logging to get useful
data out of a compiled libfprint either, so always enable debugging
output. It will still be switched off at runtime, by default.
2018-05-23 19:11:02 +02:00
Bastien Nocera
63e5d56441
build: Always allow switching log level at runtime
...
There are no parts of libfprint that are so resource intensive that we'd
want to disable logging. This avoids (hopefully rare) cases where
compiled versions of libfprint are distributed with logging completely
disabled, and thus can't be debugged.
2018-05-23 19:10:58 +02:00
Bastien Nocera
e9bfd943fc
docs: Mark a few FP_VERIFY_MATCH as constant
...
Add "%" prefix to mark it as a constant in the docs.
2018-05-23 15:18:34 +02:00
Bastien Nocera
614e2286c2
docs: Fix typo in fp_enroll_finger_img() API docs
...
Missing ":" after argument name.
2018-05-23 15:18:34 +02:00
Bastien Nocera
768a74c4bf
docs: Remove transfer information from fp_dev_open()
...
There's no GObject usage in the public API.
2018-05-23 15:18:34 +02:00
Bastien Nocera
612e9e11de
docs: Document async function callbacks
2018-05-23 15:18:34 +02:00
Bastien Nocera
06c72d54be
poll: Add missing API docs for polling functions
2018-05-23 15:18:34 +02:00