search

hughperkins / EasyCL

Easy to run kernels using OpenCL
Other
183 stars 52 forks source link

Changed to use LoadLibraryA #5

Closed hksonngan closed 9 years ago

hughperkins commented 9 years ago

Hi. Latest branch is 'copy-to-host-using-write-buffer'. When you switch to this branch, you will see that this directory is now a git submodule that points to http://github.com/hughperkins/clew . Therefore, please can you submit your PR to http://github.com/hughperkins/clew? By the way, can you provide a couple of examples of how the old code fails and the new code passes.

hksonngan commented 9 years ago

Hi, I add ADD_DEFINITIONS(-DUNICODE) ADD_DEFINITIONS(-D_UNICODE) in cmake file and compiling in VS 2013 then I got waning: E:\local\clew-master\src\clew.c(170): warning C4133: 'function' : incompatible types - from 'const char *' to 'LPCWSTR' run testclew. I got module == null and Windows error code 126 mean that "The specified module could not be found.\r\n" but change to use LoadLibraryA, it pass this test.

hughperkins commented 9 years ago

Ah I see. Got it. Sounds reasonable.