Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ins (APDU instruction codes) enum #33

Merged
merged 1 commit into from
Nov 26, 2019
Merged

Ins (APDU instruction codes) enum #33

merged 1 commit into from
Nov 26, 2019

Conversation

tarcieri
Copy link
Collaborator

Converts a bag of constant values (YKPIV_INS_*) into an enum representing APDU instruction codes (a.k.a. ins).

Among other things, this makes the Debug output for APDU more human meaningful, since it can print a text label for the instruction rather than a code number, which is helpful in trace debugging.

Converts a bag of constant values (`YKPIV_INS_*`) into an enum
representing APDU instruction codes (a.k.a. `ins`).

Among other things, this makes the `Debug` output for `APDU` more human
meaningful, since it can print a text label for the instruction rather
than a code number, which is helpful in trace debugging.
@tarcieri tarcieri merged commit ecea008 into develop Nov 26, 2019
@tarcieri tarcieri deleted the ins-enum branch November 26, 2019 18:05
@tarcieri tarcieri mentioned this pull request Dec 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant