bug#73412: 31.0.50; Improve sql-read-product

[Eli Zaretskii <eliz@gnu.org>]

> From: Stephen Berman <stephen.berman@gmx.net>
> Cc: 73412-done@debbugs.gnu.org
> Date: Sat, 21 Sep 2024 22:29:54 +0200
> 
> On Sat, 21 Sep 2024 18:21:03 +0300 Eli Zaretskii <eliz@gnu.org> wrote:
> 
> >> Likewise, `M-x sql-set-product RET' produces the same minbuffer display,
> >> requiring deletion to change the product.
> >>
> >> The reason "ansi" is displayed after the prompt is that the definition
> >> of sql-read-product uses the same non-nil value of the deprecated
> >> argument INITIAL-INPUT as is used for the argument DEF.  The attached
> >> patch makes INITIAL-INPUT nil, thus using only DEF, and also wraps the
> >> prompt argument of sql-read-product in format-prompt, so the default
> >> value is shown as part of the prompt and changing it does not require
> >> deletion.  The patch also adjusts the two callers of sql-read-product
> >> accordingly.
> >
> > Thanks.
> >
> > I think this should go to the emacs-30 branch, no?
> 
> I agree and did so in commit c1f2501f55d and am closing the bug.  Since
> this is a minor UI update and improvement with no change in
> functionality I assume it does not need to be called out in NEWS, right?

No, I don't think this warrants a NEWS item, no.