On 2/18/26 9:23 PM, Etsuro Fujita wrote:
> I think the former part of the comment should be: Return *false* if
> the password_required is defined and false for this user mapping,
> otherwise *true*.
I feel the wording of the comment is pretty awkward both before and
after your correctness fix. I am not a native speaker but shouldn't it
be something like the below which explains better what is actually going on.
/*
* Checks the value of password_required, defaults to true
* if not defined. The mapping has been pre-validated.
*/
Andreas