On 2/12/25 7:17 AM, Andrew MacLeod wrote:
The patch is mostly fine, although you probably want to change the
condition to check for a non-null stmt as well... ie
Thanks for the reminder. I saw that defaulting and made a mental note
to test that we actually had a statement, then promptly forgot about it.
Spinning it now...
I defer to the release managers about whether it goes in trunk now or
stage 1 :-)
I think this easily meets the current stage's criteria. So if you're
comfortable with the technical bits (with the adjustment noted above),
then I'll take the responsibility for "in or out of this release" decision.
Jeff
