NetBSD-Bugs archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: lib/56647: Fix misbehaviour in libedit support for readline
The following reply was made to PR lib/56647; it has been noted by GNATS.
From: Walter Lozano <walter.lozano%collabora.com@localhost>
To: gnats-bugs%netbsd.org@localhost, lib-bug-people%netbsd.org@localhost, gnats-admin%netbsd.org@localhost,
netbsd-bugs%netbsd.org@localhost
Cc:
Subject: Re: lib/56647: Fix misbehaviour in libedit support for readline
Date: Mon, 31 Jan 2022 10:14:44 -0300
On 1/29/22 17:55, Christos Zoulas wrote:
> The following reply was made to PR lib/56647; it has been noted by GNATS.
>
> From: Christos Zoulas <christos%zoulas.com@localhost>
> To: gnats-bugs%netbsd.org@localhost
> Cc: lib-bug-people%netbsd.org@localhost,
> gnats-admin%netbsd.org@localhost,
> netbsd-bugs%netbsd.org@localhost,
> walter.lozano%collabora.com@localhost
> Subject: Re: lib/56647: Fix misbehaviour in libedit support for readline
> Date: Sat, 29 Jan 2022 15:53:03 -0500
>
> --Apple-Mail=_2EE391CB-C248-473A-A50A-B4FD774C36C5
> Content-Transfer-Encoding: quoted-printable
> Content-Type: text/plain;
> charset=us-ascii
>
> Committed!
Thank you!
Regards,
Walter
>
> christos
>
> > On Jan 26, 2022, at 11:40 AM, Walter Lozano =
> <walter.lozano%collabora.com@localhost> wrote:
> >=20
> > The following reply was made to PR lib/56647; it has been noted by =
> GNATS.
> >=20
> > From: Walter Lozano <walter.lozano%collabora.com@localhost>
> > To: Christos Zoulas <christos%zoulas.com@localhost>, gnats-bugs%netbsd.org@localhost
> > Cc: lib-bug-people%netbsd.org@localhost, gnats-admin%netbsd.org@localhost, =
> netbsd-bugs%netbsd.org@localhost
> > Subject: Re: lib/56647: Fix misbehaviour in libedit support for =
> readline
> > Date: Wed, 26 Jan 2022 13:35:36 -0300
> >=20
> > This is a multi-part message in MIME format.
> > --------------i36T0iK89PRoJ4mZNCW6gGut
> > Content-Type: text/plain; charset=3DUTF-8; format=3Dflowed
> > Content-Transfer-Encoding: 7bit
> >=20
> > Hi Christos,
> >=20
> > On 1/24/22 15:28, Christos Zoulas wrote:
> >> Hi Walter, thanks for taking the time to write a reproducer. Just =
> attach it to a mail message
> >> and it will be added to this request. In the meantime I will commit =
> the changes.
> >=20
> > Here is the test program I use to test my changes. I this based on and
> > example I found and in the code found in connman [1][2] and bluez =
> [3][4].
> >=20
> > Unfortunately I still have issues, the message is not rendered =
> properly.
> > With the example I attach the output is
> >=20
> > WOOP> Typingwarning something needs your attentionTypingWOOP> Typing
> >=20
> > but it shoud be
> >=20
> > WOOP> Typing
> > warning something needs your attention
> > WOOP> Typing
> >=20
> > However at least now the message is seeing and the test typed is =
> restored.
> >=20
> > I'll keep trying to improve it as I still have time, but I'm not sure =
> if
> > I will be able to have a better solution.
> >=20
> > Please note that my test are done under Apertis (Linux Debian
> > derivative), but my understanding is that this type of changes should
> > apply without issues on NetBSD.
> >=20
> > Regards,
> >=20
> > Walter
> >=20
> > [1] https://salsa.debian.org/debian/connman
> > [2] https://gitlab.apertis.org/pkg/connman
> > [3] https://salsa.debian.org/bluetooth-team/bluez
> > [4] https://gitlab.apertis.org/pkg/bluez
> >=20
> >> Best,
> >>=20
> >> christos
> >>=20
> >=20
> > --
> > Walter Lozano
> > Collabora Ltd.
> > --------------i36T0iK89PRoJ4mZNCW6gGut
> > Content-Type: text/x-csrc; charset=3DUTF-8; name=3D"test.c"
> > Content-Disposition: attachment; filename=3D"test.c"
> > Content-Transfer-Encoding: base64
> >=20
> > =
> Ly8gRXhhbXBsZSBmcm9tIGh0dHA6Ly93d3cubWNsZC5jby51ay9ibG9nLzIwMDkvc2ltcGxl
> > =
> LWdudS1yZWFkbGluZS1jYWxsYmFjay1zdHlsZS1leGFtcGxlLmh0bWwKCiNpbmNsdWRlIDxz
> > =
> dGRpby5oPgojaW5jbHVkZSA8cmVhZGxpbmUvcmVhZGxpbmUuaD4KI2luY2x1ZGUgPHJlYWRs
> > =
> aW5lL2hpc3RvcnkuaD4KI2luY2x1ZGUgPHN0ZGxpYi5oPiAgLyogZm9yIGZyZWUoKSAqLwoj
> > =
> aW5jbHVkZSA8dW5pc3RkLmg+IC8qIGZvciB1c2xlZXAoKSAqLwojaW5jbHVkZSA8cHRocmVh
> > =
> ZC5oPgojaW5jbHVkZSA8c3RyaW5nLmg+CgpjaGFyIHJ1bm5pbmcgPSAxOwoKLy8gVGhlIGZ1
> > =
> bmN0aW9uIHRoYXQnbGwgZ2V0IHBhc3NlZCBlYWNoIGxpbmUgb2YgaW5wdXQKdm9pZCBteV9y
> > =
> bGhhbmRsZXIoY2hhciogbGluZSk7CnZvaWQgbXlfcmxoYW5kbGVyKGNoYXIqIGxpbmUpewog
> > =
> ICAgICAgIGlmKGxpbmU9PU5VTEwpewogICAgICAgICAgICAgICAgLy8gQ3RybC1EIHdpbGwg
> > =
> YWxsb3cgdXMgdG8gZXhpdCBuaWNlbHkKICAgICAgICAgICAgICAgIHByaW50ZigiXG5OVUxM
> > =
> QlVSR0VSXG4iKTsKICAgICAgICAgICAgICAgIHJ1bm5pbmcgPSAwOwogICAgICAgIH1lbHNl
> > =
> ewogICAgICAgICAgICAgICAgaWYoKmxpbmUhPTApewogICAgICAgICAgICAgICAgICAgICAg
> > =
> ICAgIC8vIElmIGxpbmUgd2Fzbid0IGVtcHR5LCBzdG9yZSBpdCBzbyB0aGF0IHVwYXJyb3cg
> > =
> cmV0cmlldmVzIGl0CiAgICAgICAgICAgICAgICAgICAgICAgIGFkZF9oaXN0b3J5KGxpbmUp
> > =
> OwogICAgICAgICAgICAgICAgfQogICAgICAgICAgICAgICAgICAgICAgICBwcmludGYoIllv
> > =
> dXIgaW5wdXQgd2FzOlxuJXNcbiIsIGxpbmUpOwogICAgICAgICAgICAgICAgICAgICAgICBm
> > =
> cmVlKGxpbmUpOwogICAgICAgIH0KfQoKLy8jZGVmaW5lIFVTRV9QUklOVEYKCnZvaWQgKiBw
> > =
> cmludF93YXJuaW5nKHZvaWQgKiBub3RoaW5nKXsKICAgICAgICBpbnQgc2F2ZWRfcG9pbnQ7
> > =
> CiAgICAgICAgY2hhciAqIHNhdmVkX2xpbmU7CgogICAgICAgIHVzbGVlcCg1MDAwMDAwKTsK
> > =
> I2lmZGVmIFVTRV9QUklOVEYKICAgICAgICBwcmludGYoIndhcm5pbmcgc29tZXRoaW5nIG5l
> > =
> ZWRzIHlvdXIgYXR0ZW50aW9uXG4iKTsKICAgICAgICBybF9yZWRpc3BsYXkoKTsKICAgICAg
> > =
> ICB1c2xlZXAoMTAwMDAwMCk7CiNlbHNlCiAgICAgICAgc2F2ZWRfcG9pbnQgPSBybF9wb2lu
> > =
> dDsKICAgICAgICBybF9zYXZlX3Byb21wdCgpOwogICAgICAgIHNhdmVkX2xpbmUgPSBybF9j
> > =
> b3B5X3RleHQoMCwgcmxfZW5kKTsKICAgICAgICBybF9yZXBsYWNlX2xpbmUoIiIsIDApOwog
> > =
> ICAgICAgIHJsX21lc3NhZ2UoIndhcm5pbmcgc29tZXRoaW5nIG5lZWRzIHlvdXIgYXR0ZW50
> > =
> aW9uXG4iKTsKICAgICAgICBybF9yZWRpc3BsYXkoKTsKICAgICAgICAvL3VzbGVlcCgxMDAw
> > =
> MDAwKTsKICAgICAgICBybF9yZXN0b3JlX3Byb21wdCgpOwogICAgICAgIHJsX3JlcGxhY2Vf
> > =
> bGluZShzYXZlZF9saW5lLCAwKTsKICAgICAgICBybF9wb2ludCA9IHNhdmVkX3BvaW50Owog
> > =
> ICAgICAgIHJsX3JlZGlzcGxheSgpOwojZW5kaWYKICAgICAgICByZXR1cm4gTlVMTDsKfQoK
> > =
> Ly8gVGhlIG1haW4gZW50cnktcG9pbnQgZm9yIHRoZSBwcm9ncmFtCmludCBtYWluKCkKewog
> > =
> ICAgICAgIGNvbnN0IGNoYXIgKnByb21wdCA9ICJXT09QPiAiOwogICAgICAgIHB0aHJlYWRf
> > =
> dCB0aHJlYWRfaWQ7CgogICAgICAgIC8vIEluc3RhbGwgdGhlIGhhbmRsZXIKICAgICAgICBy
> > =
> bF9jYWxsYmFja19oYW5kbGVyX2luc3RhbGwocHJvbXB0LCAocmxfdmNwZnVuY190KikgJm15
> > =
> X3JsaGFuZGxlcik7CgogICAgICAgIC8vIEVudGVyIHRoZSBldmVudCBsb29wIChzaW1wbGUg
> > =
> ZXhhbXBsZSwgc28gaXQgZG9lc24ndCBkbyBtdWNoIGV4Y2VwdCB3YWl0KQogICAgICAgIHJ1
> > =
> bm5pbmcgPSAxOwoKICAgICAgICBwdGhyZWFkX2NyZWF0ZSgmdGhyZWFkX2lkLCBOVUxMLCAm
> > =
> cHJpbnRfd2FybmluZywgTlVMTCk7CgogICAgICAgIHdoaWxlKHJ1bm5pbmcpewogICAgICAg
> > =
> ICAgICAgICAgLy9wcmludGYoIndhaXRpbmdcbiIpOwogICAgICAgICAgICAgICAgdXNsZWVw
> > =
> KDEwMDAwKTsKICAgICAgICAgICAgICAgIHJsX2NhbGxiYWNrX3JlYWRfY2hhcigpOwogICAg
> > =
> ICAgIH07CiAgICAgICAgcHJpbnRmKCJcbkV2ZW50IGxvb3AgaGFzIGV4aXRlZFxuIik7Cgog
> > =
> ICAgICAgIC8vIFJlbW92ZSB0aGUgaGFuZGxlcgogICAgICAgIHJsX2NhbGxiYWNrX2hhbmRs
> > ZXJfcmVtb3ZlKCk7CgogICAgICAgIHJldHVybiAwOyAvLyBoYXBweSBlbmRpbmcKfQo=3D
> >=20
> > --------------i36T0iK89PRoJ4mZNCW6gGut--
> >=20
>
>
> --Apple-Mail=_2EE391CB-C248-473A-A50A-B4FD774C36C5
> Content-Transfer-Encoding: 7bit
> Content-Disposition: attachment;
> filename=signature.asc
> Content-Type: application/pgp-signature;
> name=signature.asc
> Content-Description: Message signed with OpenPGP
>
> -----BEGIN PGP SIGNATURE-----
> Comment: GPGTools - http://gpgtools.org
>
> iF0EARECAB0WIQS+BJlbqPkO0MDBdsRxESqxbLM7OgUCYfWpLwAKCRBxESqxbLM7
> OpJKAKCEq8iUZbXjQbic0yCo6TXOBs9BxgCfaorsuaD373jDnocEv1EZ+k0qxUM=
> =++SY
> -----END PGP SIGNATURE-----
>
> --Apple-Mail=_2EE391CB-C248-473A-A50A-B4FD774C36C5--
>
--
Walter Lozano
Collabora Ltd.
Home |
Main Index |
Thread Index |
Old Index