sgminer-dev / sgminer

Scrypt GPU miner
GNU General Public License v3.0
631 stars 825 forks source link

Build error introduced in recent commit #190

Closed OhGodAPet closed 10 years ago

OhGodAPet commented 10 years ago
sgminer.c: In function ‘api_thread’:
sgminer.c:5001:13: error: incompatible types when assigning to type ‘pthread_t’ from type ‘long int’
  PTH(mythr) = 0L;

EDIT: Found one more instance:

sgminer.c: In function ‘kill_mining’:
sgminer.c:3185:23: error: invalid operands to binary != (have ‘pthread_t’ and ‘long int’)
   if (thr && PTH(thr) != 0L)
Bllacky commented 10 years ago

Hmm, the last commit I have checked and compiled with VC++ . Let me see if I get that error as well.

OhGodAPet commented 10 years ago

I used GCC 4.8.2.

Bllacky commented 10 years ago

Ok, none of the changes proposed by my commit seem to have any influence on your errors.

basically GCC is complaining here:

        thr = get_thread(i);
        if (thr && PTH(thr) != 0L)
            pth = &thr->pth;

AND HERE

    api(api_thr_id);

    PTH(mythr) = 0L;
Bllacky commented 10 years ago

The only one could have been

But this change has already been reverted.

Just re-downloaded the latest version from GIT and compiled it again. Seems to be working fine. This is beyond me, sorry.

OhGodAPet commented 10 years ago

Fixed now.