`parameter_assignments` false positive

[FMorschel]

Repro:

void pattern(bool value) {
  value = true; // parameter_assignments 
  if (!value || value) {}
  //   ^  parameter_assignments
}

The !value triggers parameter_assignments here, and it shouldn't.

Really not sure what it was trying to do there. At least it only triggers if we have a real case for it. It also happens if you exchange the operands (but we get a dead code warning and that is not a case I wanted to use as a repro) so it is not the ordering, it has something to do with !.

FYI @bwilkerson