From eae5721f8e8103ce0d46d28b17e0e7f354690f7c Mon Sep 17 00:00:00 2001 From: Bastien Nocera Date: Thu, 29 Nov 2018 17:18:16 +0100 Subject: [PATCH] vfs101: Add FIXME about fpi_imgdev_set_action_result usage We shouldn't need to use fpi_imgdev_set_action_result(), because as fpi_imgdev_image_captured() is already there. --- libfprint/drivers/vfs101.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/libfprint/drivers/vfs101.c b/libfprint/drivers/vfs101.c index 32c6739..8faaa23 100644 --- a/libfprint/drivers/vfs101.c +++ b/libfprint/drivers/vfs101.c @@ -767,6 +767,11 @@ img_extract(fpi_ssm *ssm, /* Notify image captured */ fpi_imgdev_image_captured(dev, img); + /* FIXME + * What is this for? The action result, and the enroll stages should + * already be handled in fpi_imgdev_image_captured() + */ + /* Check captured result */ if (fpi_imgdev_get_action_result(dev) >= 0 && fpi_imgdev_get_action_result(dev) != FP_ENROLL_RETRY &&