mattwparas / steel / 19991502472

45%

Build Type

push

github

Committed by web-flow

Commit Message

make it possible to Hash all steel values (#580)

* don't make different reducers equal?

previously `(equal? (into-sum) (into-count))` returned true, even though
i don't think that makes a lot of sense.

* make it possible to compare Boxed values

`visit_boxed_value` was already implemented, it just wasn't called.

before:

```
λ > (equal? (box-strong 0) (box-strong 0))
=> #false
```

now:

```
λ > (equal? (box-strong 0) (box-strong 0))
=> #true
```

* make futures / futurefuncs / streams comparable with equal?

`eqv?` already worked on futures, futurefuncs and streams by just doing
ptr_eq on the Gc.

* make ports be able to `equal?` eachother if they are the same instance

* implement Hash for all possible SteelVals

* remove `SteelVal::is_hashable` and all of its usages.

* remove all references to `hashable?` in the documentation

* add some hashing tests

Run Details

46 of 90 new or added lines in 5 files covered. (51.11%)

19 existing lines in 4 files now uncovered.

16017 of 32407 relevant lines covered (49.42%)

3661430.98 hits per line