Skip to content

Suggested small correction#3

Open
Y-Innovate wants to merge 1 commit intocicsdev:mainfrom
Y-Innovate:feature/smallfix
Open

Suggested small correction#3
Y-Innovate wants to merge 1 commit intocicsdev:mainfrom
Y-Innovate:feature/smallfix

Conversation

@Y-Innovate
Copy link

Hi Ian,
I came across your repo searching for a way to Base64 encode and decode in Assembler and your code is simply wonderful. I very much appreciate you sharing this freely on Github.
I ran into what I think is a little bug and made a suggestion in the form of a pull request. I'm not 100% sure, but I think it's better in base64d.asm to test the yet undecoded character in R1 for '=' instead of the decoded char in R3.
Cheers!
Bobby

Signed-off-by: Bobby Tjassens Keiser <bobby.tjassens.keiser@y-innovate-it.com>
@ian-burnett
Copy link
Member

Hi @Y-Innovate

Thanks for the input. Let me page this repo back into memory and I'll take a look later this week.

Ian

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

Comments