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

Add a Unicode test to strings/rabin_karp.py #1067

Closed
cclauss opened this issue Jul 24, 2019 · 1 comment
Closed

Add a Unicode test to strings/rabin_karp.py #1067

cclauss opened this issue Jul 24, 2019 · 1 comment
Labels
awaiting changes A maintainer has requested changes to this PR

Comments

@cclauss
Copy link
Member

cclauss commented Jul 24, 2019

@mahbubcseju

Something like:

    pattern = "Lü"
    text = "Lüsai"
    assert rabin_karp(pattern, text)
    pattern = "Lue"
    assert not rabin_karp(pattern, text)

Plus maybe some emojis 😄 🎄 🍰

@cclauss cclauss added the awaiting changes A maintainer has requested changes to this PR label Jul 24, 2019
qckzr added a commit to qckzr/Python that referenced this issue Aug 5, 2019
cclauss pushed a commit that referenced this issue Aug 5, 2019
* Added print function into matrix_multiplication_addition.py and removed blank space in data_structures/binary tree directory

* Removed .vs/ folder per #893

* Rename matrix_multiplication_addition.py to matrix_operation.py

* Unicode test on strings/rabin_karp.py per #1067
@github-actions
Copy link

Stale issue message

@github-actions github-actions bot closed this as completed Dec 7, 2019
stokhos pushed a commit to stokhos/Python that referenced this issue Jan 3, 2021
* Added print function into matrix_multiplication_addition.py and removed blank space in data_structures/binary tree directory

* Removed .vs/ folder per TheAlgorithms#893

* Rename matrix_multiplication_addition.py to matrix_operation.py

* Unicode test on strings/rabin_karp.py per TheAlgorithms#1067
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting changes A maintainer has requested changes to this PR
Projects
None yet
Development

No branches or pull requests

1 participant