search

KhronosGroup / OpenCL-SDK

OpenCL SDK
Apache License 2.0
579 stars 120 forks source link

add enumopencl, copybuffer, and copybufferkernel samples #13

Closed bashbaug closed 4 years ago

bashbaug commented 4 years ago

This is a continuation of #6, which was closed when the default branch was renamed.

I have integrated all review feedback. @kpet and @jrprice please take another look here - thanks!

jrprice commented 4 years ago

@jrprice, please see that I have added error checking as you requested.

I don't see this change, is there a commit missing?

bashbaug commented 4 years ago

I don't see this change, is there a commit missing?

Sorry about that, I missed the most important commit with review feedback. It's added now.

bashbaug commented 4 years ago

Thanks! I've integrated the last round of review feedback. I also added a basic AppVeyor config file for Windows CI but it doesn't seem to be triggering yet, perhaps because this is a private repo?

kpet commented 4 years ago

P.S.: I think it's actually already enabled. Merging.