PDA

View Full Version : Proposed change to C++ bindings



clamport
02-20-2010, 03:25 PM
I have been playing around with OpenGL interop, and saw that the cl::Context constructor had properties set as NULL. From all of the OpenGL example code I have seen this parameter is for relaying OpenGL information to OpenCL, should this be changed (cl.hpp, line 1486) from:

Context(
const VECTOR_CLASS<Device>& devices,
cl_context_properties* properties = NULL,
void (*notifyFptr)(
const char *,
const void *,
::size_t,
void *) = NULL,
void* data = NULL,
cl_int* err = NULL)

to


Context(
const VECTOR_CLASS<Device>& devices,
cl_context_properties* properties,
void (*notifyFptr)(
const char *,
const void *,
::size_t,
void *) = NULL,
void* data = NULL,
cl_int* err = NULL)

Thanks,
Chris

coleb
02-23-2010, 05:02 PM
Why change this? Is NULL not a reasonable default. The user can always specify another set of context properties.

clamport
02-23-2010, 07:44 PM
Nevermind, thought it was assignment.