17:17:41 <tevador> sech1: please review https://github.com/tevador/RandomX/pull/179
17:25:19 <cohcho> I've added exactly this function for case when miner calculates the last nonce
17:25:47 <cohcho> but i didn't change "uint32_t chan" to "int chain", maybe it's needed
17:28:08 <tevador> it's the same unless RANDOMX_PROGRAM_COUNT exceeds INT_MAX, which won't happen
17:30:25 <tevador> btw, with batching, the browser miner will have ~95% of native performance
17:47:45 <sech1> tevador looks good, but naming of these functions can be improved I think. It's not obvious that output in *_next function is for the previous hash. But this is for another PR.
17:48:19 <sech1> maybe people actually read comments in the code, I don't know
17:51:45 <cohcho> I just did few recompilation until got why hashes from _first/_next doesn't match with _hash and reordered some lines of code
18:58:43 <tevador> sech1: yes, I got an email the other day, some guy was asking why his hashes don't match - guess why, his nonce was one off because of how randomx_calculate_hash_next works