Use a signed type for NodeToSplitPlayer

Whether char is signed or not is implementation dependent, so this fixes UB for the return -1 case on platforms where char is unsigned by default like non-apple arm(64).