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

Added the Julia support for the StdDequeIterator #431

Closed
wants to merge 45 commits into from

Conversation

abdoei
Copy link
Contributor

@abdoei abdoei commented May 4, 2024

Due to the testjll is not up-to-date with the main, my changes can be found in the test/stdlib.jl:361 and in the src/StdLib.jl:201

kleinschmidt and others added 30 commits October 4, 2023 08:21
* Improve UTF-8 support for StdString

* Common support for StdWString/StdString

* Fix invalid empty character literal on Julia 1.6

* Debug Windows StdWString issue

* Fix StdWString tests on Windows

* Support constructing of malformed chars for tests

* Update iterating into invalid index

* Support null-terminated constructor via Cstring

* Comment on custom iterate method

* Add docstring for StdString(::Any, ::Integer)

* Add tests for StdString(::String, ::Integer)

* Add README entry

* fixup! Add README entry

* fixup! Add tests for StdString(::String, ::Integer)
* Refactor StdVector tests

* Support parameterized constructors

* Support StdVector{Any}

* Refactor tests

* Refactor logic in StdVector(::Vector)

* Empty array constructor tests

* Use eltype
@barche
Copy link
Collaborator

barche commented May 10, 2024

This PR should be rebased in such a way that it only reflects your changes, there is a very long list of commits now.

@abdoei abdoei closed this May 12, 2024
@abdoei
Copy link
Contributor Author

abdoei commented May 12, 2024

This testjll branch is not up to date with the main so I had to make it so. I think you could review my changes from my last commit. Is there something to get the same output in the way you want I can't understand?

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.

8 participants