Patrick Steinhardt <ps@xxxxxx> writes: > The `initial_transaction` flag is tracked as a signed integer, but we > typically pass around flags via unsigned integers. Adapt the type > accordingly. It would not matter while the information in the variable is 1 bit and the check is "is the whole variable 0?", which is what the current code does on this parameter. But when we'd start using the parameter as a flag word to stuff more bits, this starts to matter. Good future-proofing. Thanks.