aboutsummaryrefslogtreecommitdiff
path: root/TODO.md
diff options
context:
space:
mode:
authorRobin Krahl <robin.krahl@ireas.org>2019-01-16 02:52:58 +0000
committerRobin Krahl <robin.krahl@ireas.org>2019-01-16 20:17:07 +0100
commit3c8092cd937d6f449b1959eab9e7e15549970d85 (patch)
tree611be4c031b01ff8143a677e2954d830146ecde3 /TODO.md
parent7c2bb330b125cde7ceb55616b3d8c50a98840c58 (diff)
downloadnitrokey-rs-3c8092cd937d6f449b1959eab9e7e15549970d85.tar.gz
nitrokey-rs-3c8092cd937d6f449b1959eab9e7e15549970d85.tar.bz2
Fix result_from_string for empty return values
An empty string returned from a libnitrokey function can either indicate an error or be a valid return value. Previously, we assumed that it indicates an error. But instead, we should check the last command status and use it to decide whether to return the empty string or an error code. This breaks the unit tests that assume that empty strings cause errors. These will be fixed in the next patches.
Diffstat (limited to 'TODO.md')
-rw-r--r--TODO.md1
1 files changed, 0 insertions, 1 deletions
diff --git a/TODO.md b/TODO.md
index 7d7ff1a..28bd3b8 100644
--- a/TODO.md
+++ b/TODO.md
@@ -11,7 +11,6 @@
- Find a nicer syntax for the `write_config` test.
- Prevent construction of internal types.
- More specific error checking in the tests.
-- Differentiate empty strings and errors (see `result_from_string`).
- Check integer conversions.
- Consider implementing `Into<CommandError>` for `(Device, CommandError)`
- Lock password safe in `PasswordSafe::drop()` (see [nitrokey-storage-firmware