Fix for texture/unwrap - Githubissues

ztellman / penumbra

not under active development - idiomatic opengl bindings for clojure

354 stars 43 forks source link

Closed zoka closed 8 years ago

zoka commented 13 years ago

Hi Zach,

Could you please review Penumbra patch at https://gist.github.com/1102320

Changes:

texture/unwrap was returning a typed byte buffer instead of array, so helper function buffer-to-array was added. This bug caused number of tests in gpgpu.clj to fail
- there were couple of typos in Penumbra test code (app/periodic-update! and app/start)
partition-all and flatten need not be specified since they have been moved to core

Regards Zoka

ztellman commented 13 years ago

It looks good. Once I get a chance to do a quick sanity check, I'll merge the changes.