On Sat, Jan 19, 2013 at 3:43 PM, magnum <notifications@github.com> wrote:

> In src/common_opencl_pbkdf2.c:
>
> > @@ -369,6 +372,8 @@ static gpu_mem_buffer exec_pbkdf2(cl_uint *pass_api,cl_uint *salt_api,cl_uint sa
> >
> >  		HANDLE_CLERROR(clFinish(cmdq[platform_no][dev_no]), "clFinish error");
> >
> > +		HANDLE_CLERROR(clFinish(cmdq[platform_no][dev_no]), "clFinish error");															
> > +
> >  		clGetEventProfilingInfo(evnt, CL_PROFILING_COMMAND_SUBMIT, sizeof(cl_ulong), &startTime, NULL);
>
> What is the rationale for calling clFininsh twice on the same queue?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/magnumripper/JohnTheRipper/pull/176/files#r2705134>.
>
>

Thanks for pointing it out. It is a typo which happened when I copy pasted
a portion of code. Although this shouldn't be a problem , I will fix it in
my next commit.

Regards,
Sayantan