Skip to content

Using unordered map + making code more readable#18

Open
galloj wants to merge 2 commits into99x:masterfrom
galloj:patch-1
Open

Using unordered map + making code more readable#18
galloj wants to merge 2 commits into99x:masterfrom
galloj:patch-1

Conversation

@galloj
Copy link
Copy Markdown
Contributor

@galloj galloj commented Oct 19, 2019

This should solve #16 and make code more readable

@ilka-schulz
Copy link
Copy Markdown
Contributor

lgtm

This PR is very old – is there any plans to merge it?

@shalithasuranga
Copy link
Copy Markdown
Contributor

Hey.. sorry for the delayed response.. @galloj Could you please apply some fix for merge conflicts? Let's merge this soon :)

@ilka-schulz
Copy link
Copy Markdown
Contributor

I assume the merge conflict arises from #39. I could resolve the merge conflict but GitHub says I lack write access? Sorry, I am new to GitHub (I usually use GitLab), tell me if I can help...

@galloj
Copy link
Copy Markdown
Contributor Author

galloj commented Oct 26, 2021

Hope I done it correctly, the size of diff scares me.

@ilka-schulz
Copy link
Copy Markdown
Contributor

lgtm

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.

3 participants