japin <japinli(at)hotmail(dot)com> writes:
> I find that the outputstr variable in logicalrep_write_tuple() only use in
> `else` branch, I think we can narrow the scope, just like variable outputbytes
> in `if` branch (for more readable).
Agreed, done.
For context, I'm not usually in favor of making one-off stylistic
improvements: the benefit seldom outweighs the risk of creating
merge hazards for future back-patching. But in this case, the
code involved is mostly new in v14, so improving it now doesn't
cost anything in that way.
regards, tom lane