search

cbuchner1 / CudaMiner

a CUDA accelerated litecoin mining application based on pooler's CPU miner
Other
687 stars 304 forks source link

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

Open JoeSuber opened 10 years ago

JoeSuber commented 10 years ago

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

jbroome commented 10 years ago

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

brianredbeard commented 10 years ago

LGTM

:+1:

cbuchner1 commented 10 years ago

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

2014-04-13 21:05 GMT+02:00 brian notifications@github.com:

[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 .

JoeSuber commented 10 years ago

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

On Mon, Apr 14, 2014 at 9:19 AM, Christian Buchner <notifications@github.com

wrote:

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

2014-04-13 21:05 GMT+02:00 brian notifications@github.com:

[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 .