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

Sequence equality is broken #243

Open
frenchy64 opened this issue Feb 6, 2025 · 1 comment · May be fixed by #249
Open

Sequence equality is broken #243

frenchy64 opened this issue Feb 6, 2025 · 1 comment · May be fixed by #249

Comments

@frenchy64
Copy link
Contributor

frenchy64 commented Feb 6, 2025

clojure.core=> (= (range 5) (range 5))
false
clojure.core=> (= () (lazy-seq))
false


;; from clojure-test

FAIL in (nil) (nil:nil)
expected: (= (range 1) (butlast (range 2)))
  actual: (not (= (0) (0)))

FAIL in (nil) (nil:nil)
expected: (= (range 2) (butlast (range 3)))
  actual: (not (= (0 1) (0 1)))

FAIL in (nil) (nil:nil)
expected: (= (range 3) (butlast (range 4)))
  actual: (not (= (0 1 2) (0 1 2)))
@frenchy64 frenchy64 changed the title range equality is broken Sequence equality is broken Feb 6, 2025
@jeaye
Copy link
Member

jeaye commented Feb 6, 2025

I suspect this is related to #87, which is due to chunked seqs being borked. This is some very intricate C++ code which needs a better design. I have one in mind, but I'm likely the only one with enough knowledge to do it and other things have priority.

@jeaye jeaye added the type:bug label Feb 6, 2025
@frenchy64 frenchy64 linked a pull request Feb 7, 2025 that will close this issue
@jeaye jeaye removed the type:bug label Feb 18, 2025
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 a pull request may close this issue.

2 participants