On Tue, 11 Mar 2025 at 05:18, Tom Lane <t...@sss.pgh.pa.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
