CudaMiner icon indicating copy to clipboard operation
CudaMiner copied to clipboard

Update salsa_kernel.cu w extra parenthesis to allow compile on gcc

Open JoeSuber opened this issue 11 years ago • 4 comments

just a little syntax error that slipped through... gcc compile works with this change now.

JoeSuber avatar Apr 12 '14 09:04 JoeSuber

This worked for me on CentOS 6.5, x86_64. Thank you!

jbroome avatar Apr 13 '14 04:04 jbroome

LGTM

:+1:

brianredbeard avatar Apr 13 '14 19:04 brianredbeard

I removed a round bracket, instead of adding one. How's that?

2014-04-13 21:05 GMT+02:00 brian [email protected]:

[image: LGTM] http://www.lgtm.in/i/c2z1

[image: :+1:] http://www.lgtm.in/u/c2z1

— Reply to this email directly or view it on GitHubhttps://github.com/cbuchner1/CudaMiner/pull/127#issuecomment-40316345 .

cbuchner1 avatar Apr 14 '14 14:04 cbuchner1

absolutely fine. Thanks! I just didn't want to offend the parenthesis partisans :)

On Mon, Apr 14, 2014 at 9:19 AM, Christian Buchner <[email protected]

wrote:

I removed a round bracket, instead of adding one. How's that?

2014-04-13 21:05 GMT+02:00 brian [email protected]:

[image: LGTM] http://www.lgtm.in/i/c2z1

[image: :+1:] http://www.lgtm.in/u/c2z1

— Reply to this email directly or view it on GitHub< https://github.com/cbuchner1/CudaMiner/pull/127#issuecomment-40316345> .

— Reply to this email directly or view it on GitHubhttps://github.com/cbuchner1/CudaMiner/pull/127#issuecomment-40370560 .

JoeSuber avatar Apr 14 '14 16:04 JoeSuber