On Tue, 11 Mar 2025 at 05:18, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Here's a hopefully-final v3 that makes the two changes discussed.
> Now with a draft commit message, too.
Looks good to me.
The only minor points I noted down while reviewing were 1)
name_active_windows()'s newname variable could be halved in size and,
2) explain.sql's new test could name the window "w1" instead of "w" to
exercise the name selection code a bit better. Both are minor points,
but I thought I'd mention them anyway.
David