Re: mutt_buffer_concat_path is broken [PATCH: 1/1] Version 2

"Kevin J. McCarthy" <[email protected]> Sat, 9 May 2026 09:27:15 +0800
Newsgroups gmane.mail.mutt.devel
Message-ID <af6Nc9eGB63QaCO0@qinghai>
On Fri, May 08, 2026 at 08:34:28PM +0200, Matthias Andree via Mutt-dev wrote:
>Am 08.05.26 um 06:57 schrieb Kevin J. McCarthy:
>>Fortunately I don't have to choose.  I had forgotten that there is 
>>another version of the function in lib.c: mutt_concat_path(). It's 
>>not called, but it should be fixed too.  And Alex's one-line fix 
>>applies easily there.
>
>If it's not called, why not delete it?

Yeah, at the time I kept it in because it was a "library" function that 
might in theory be helpful later.

But all the path code is using buffers now, so I guess it doesn't make 
sense to keep it.

I'll put on my todo list for 2.5 development to go through lib and 
remove unused functions.

-- 
Kevin J. McCarthy
GPG Fingerprint: 8975 A9B3 3AA3 7910 385C  5308 ADEF 7684 8031 6BDA
signature.asc (application/pgp-signature, 833 B)
-----BEGIN PGP SIGNATURE-----

iQIzBAEBCgAdFiEEiXWpszqjeRA4XFMIre92hIAxa9oFAmn+jXMACgkQre92hIAx
a9pdNg/+Nqg2wTCwvqOI2WrkcSs9nmEI+TNm9OkTCsH1J7BMG6XmciSG91xcJBBa
Dub5w7HYX+app+Rno0KFVtZ2ixxmCGfrYFO2rY9/X7emJxBWJFmv0N/D8XSwa2ED
DAqCxwql0vaYIEASSM9UM0lfDa2IjdS/+kdLc3U3zgOhKVJphXuba9Wh+St/rnxW
oVDTxFTVDzlqqWO31qlM4RLDh+mBhhq+k52CAV648TlBtl32txx1ZlfmXzFIVPI3
xt820qChYJMa/DeGXp77eVTdAhZ423s6js63Geran6unpQ2rNKCsgtszRDNTM6c6
fJXeh0geZmUXVRbxm2kr5OwfRuMIIgPRD/OPbF86pdqp4nEhgw8tQ0Zxm5/KGf0M
RN/dPcJ1jTCO//5OH9SHH/cntBIXaFWo19j5JX042KtLEe1NUwIEvvdb8oDLWPTr
eaoJSMgs5zD7BysMhdzVrc468+oED+z7fRImF+qsbYf30jpDr0ptnGWit6njEjSF
dbINp7Juvi3BnrFYQepLU/gWz075b8aAOPjhP7uf3YnC42Q6P9tS2TYV3VClMob7
7G7ztCdo4GjHAy+LzKyKw1rygCe+bDyZSurzvqKcqMJGBYJEasiM+XEsY2GTOVXs
5JCIchlqDebHko3kK+vy43PmoCWXQd3qnnEzl+KNpc85RW0ku94=
=x+cT
-----END PGP SIGNATURE-----