Add comment that replies to the question in comment above

Change-Id: Iad8896df4491b2b4363fb2d16134e92dd9545c35

1 files changed, 9 insertions, 0 deletions

diff --git a/sc/source/core/opencl/openclwrapper.cxx b/sc/source/core/opencl/openclwrapper.cxx
index 3812bd752358..4516af5dc667 100644
--- a/sc/source/core/opencl/openclwrapper.cxx
+++ b/sc/source/core/opencl/openclwrapper.cxx
@@ -909,8 +909,17 @@ bool switchOpenCLDevice(const OUString* pDevice, bool bAutoSelect, bool bForceEv
     OpenCLDevice::initOpenCLAttr(&env);
 
     // why do we need this at all?
+
+    // (Assuming the above question refers to the mpArryDevsID
+    // initialisation below.) Because otherwise the code crashes in
+    // initOpenCLRunEnv(). Note that the initOpenCLAttr() call above
+    // sets mnIsUserCreated to 1, thus the code in initOpenCLRunEnv()
+    // that would initialise mpArryDevsID is not executed. Confused?
+    // You should be.
+
     OpenCLDevice::gpuEnv.mpArryDevsID = (cl_device_id*) malloc( sizeof(cl_device_id) );
     OpenCLDevice::gpuEnv.mpArryDevsID[0] = pDeviceId;
+
     return !OpenCLDevice::initOpenCLRunEnv(0);
 }