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

fix: lf hitag hts rdbl read by page #2556

Merged
merged 1 commit into from
Oct 4, 2024

Conversation

douniwan5788
Copy link
Contributor

No description provided.

Copy link

github-actions bot commented Oct 4, 2024

You are welcome to add an entry to the CHANGELOG.md as well

@iceman1001
Copy link
Collaborator

Nice!

How about you use #defines for the reasons codes.

This is the preferred style

#define PM3_REASON_xxx -4

@iceman1001 iceman1001 merged commit cd4c283 into RfidResearchGroup:master Oct 4, 2024
12 checks passed
@douniwan5788
Copy link
Contributor Author

That's my plan! But I think we should only set #define PM_REASON_UNKNOWN -1 in pm3_cmd.h. Let each command decide on their own headers for other reason values.

@iceman1001
Copy link
Collaborator

I prefer having one place where you define it. If all commands going to have their own, its gonna be a nightmare maintaining.

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.

2 participants