search

iu-parfunc / accelerack

C Bindings for Accelerate, Racket Frontend to Accelerate
2 stars 2 forks source link

Bogus acc-array->list output #43

Closed rrnewton closed 8 years ago

rrnewton commented 8 years ago

This seems odd:

> (require accelerack)
> (acc-array->list (acc-array 99))
(99)

It should be just 99, and we should rename the function acc-array->sexp.

prak5190 commented 8 years ago

@rrnewton Changed it to acc-array->sexp . Can you audit it , all tests pass - but I had to change read-data* contract to any/c instead of pair? What should the contract be for just an sexp ? https://github.com/iu-parfunc/accelerack/commit/886e09fc04bd1631552fe0726445da19dc8f2838

rrnewton commented 8 years ago

LGTM