Hi Andrew,AndrewGrant wrote: ↑Wed Sep 02, 2020 10:17 pmIts actually quite hard to see. I'de really suggest you standardize your code before your repo gets any bigger. Otherwise you'll be posting about strange bugfixes for the next half a decade.OliverBr wrote: ↑Wed Sep 02, 2020 9:45 pm Of course, it's not that difficult too see:
Code: Select all
void regPromotions(int f, int c, u64 bt, int* mlist, int* mn, int cap, int queen) { while (bt) { int t = pullLsb(&bt); Move m = f | _ONMV(c) | _PIECE(PAWN) | _TO(t) | (cap ? _CAP(identPiece(t)) : 0); if (queen) mlist[(*mn)++] = m | _PROM(QUEEN); mlist[(*mn)++] = m | _PROM(KNIGHT); mlist[(*mn)++] = m | _PROM(ROOK); mlist[(*mn)++] = m | _PROM(BISHOP); } }
some questions:
"Its actually quite hard to see." - It's really hard to see that Knights are ordered before Rook and Bishops?
"Standardize my code" - Can you please elaborate? What is non-standard in this code?
"Gets any bigger" - Do you mean the size of my source? The code didn't grow since version 3. No, even vice versa, it's getting smaller. This is one of the most important features.
"Otherwise you'll be posting about strange bugfixes for the next half a decade." - There is some discontent in this statement. What exactly am I doing to cause this? I do apologize for it.