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

Changing .to_string to .to_hex_string to fix conflict and added a Display impl for Record #12

Merged
merged 5 commits into from Jun 13, 2020

Conversation

skrapi
Copy link
Contributor

@skrapi skrapi commented Jun 11, 2020

I have done some clean up using clippy recommendations.

I changed .to_string to .to_hex_string to prevent it conflicting with the .to_string method which I think is part of the std library.

I added a Display impl for Record which outputs the addresses in HEX to help with address debugging.

@skrapi skrapi mentioned this pull request Jun 11, 2020
@coveralls
Copy link

Coverage Status

Coverage increased (+0.7%) to 95.878% when pulling cf2ee8d on skrapi:master into fadf712 on martinmroz:master.

@martinmroz
Copy link
Owner

Hey, I just wanted to say thank you for contributing your fix! I'm really happy you've been able to put this to good use. I'd love to hear more about what you're building with it.

@martinmroz martinmroz merged commit bec73b5 into martinmroz:master Jun 13, 2020
@martinmroz
Copy link
Owner

Since this breaks API, I'll probably have to bump the major version. While I'm in there, I'll take a pass at cleanup/modernization, and see if I can't have a new crate version uploaded by EOD tomorrow.

@skrapi
Copy link
Contributor Author

skrapi commented Jun 13, 2020

Great thank you. I am using this to strip a hex file, creating a binary app image which I can use for bootloading a IoT device. I really appreciate the work you have put in. It’s helped me a lot.

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.

None yet

3 participants