Closed basilhussain closed 2 years ago
Thanks for the commits! Very nice, I'll check these out this weekend.
Yes, there were several bugs I didn't try to fix before uploading. It was working for my purposes already and just wanted to get it up on github.
Heh, I forgot that GitHub tracks head of the branch for pull request, so I guess you also get all the other bug fixes and improvements I made since originally filing the pull request. :-)
Additional stuff fixes the bugs I reported in issues #10 and #11, as well as adding IHEX and SREC support (because I wanted to be able to run some existing code that wasn't in ELF format). By the way, with that you will see I deleted a bunch of the unused code in elf.c to clear clutter so I could focus on the rest of the code - hope you don't mind, feel free to restore it.
While you're looking at this stuff, could you please also address licensing (as raised in issue #6)? I'm a bit hesitant to make any future contributions while the licensing situation of the code is unknown.
This pull request fixes the compilation and linking errors and warnings, as mentioned in issue #5.
I also added a .gitignore file that excludes all Visual Studio-related fluff from the repo, hope that's okay.