Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 16 Sep 2019 09:08:06 +0200
From:      Baptiste Daroussin <bapt@FreeBSD.org>
To:        Dimitry Andric <dim@FreeBSD.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r352341 - head/contrib/libedit
Message-ID:  <20190916070805.ykasrsmqdol3jwqt@ivaldir.net>
In-Reply-To: <201909142149.x8ELnhqE035300@repo.freebsd.org>
References:  <201909142149.x8ELnhqE035300@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help

--zzllgpaayqdv4sjx
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Sat, Sep 14, 2019 at 09:49:43PM +0000, Dimitry Andric wrote:
> Author: dim
> Date: Sat Sep 14 21:49:42 2019
> New Revision: 352341
> URL: https://svnweb.freebsd.org/changeset/base/352341
>=20
> Log:
>   Fix arm and aarch64 builds of libedit after r352275
>  =20
>   On arm and arm64, where chars are unsigned by default, buildworld dies
>   with:
>  =20
>   --- terminal.o ---
>   /usr/src/contrib/libedit/terminal.c:569:41: error: comparison of
>   integers of different signs: 'wint_t' (aka 'int') and 'wchar_t' (aka
>   'unsigned int') [-Werror,-Wsign-compare]
>                                        el->el_cursor.v][where & 0370] !=3D
>                                        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^
>   /usr/src/contrib/libedit/terminal.c:659:28: error: comparison of
>   integers of different signs: 'wint_t' (aka 'int') and 'wchar_t' (aka
>   'unsigned int') [-Werror,-Wsign-compare]
>                                        [el->el_cursor.h] =3D=3D MB_FILL_C=
HAR)
>                                        ~~~~~~~~~~~~~~~~~ ^  ~~~~~~~~~~~~
>  =20
>   Fix this by making MB_FILL_CHAR a wint_t, so no casting is needed.
>  =20
>   Note that in https://reviews.freebsd.org/D21584 this was also proposed
>   by Yuichiro Naito <naito.yuichiro_gmail.com>.
>  =20
>   Reviewed by:	bapt
>   Subscribers:	naito.yuichiro_gmail.com, ml_vishwin.info
>   MFC after:	3 weeks
>   X-MFC-With:	r352275
>   Differential Revision: https://reviews.freebsd.org/D21657
>=20
Just for completeness it was also approved by christos@NetBSD (upstream)

--zzllgpaayqdv4sjx
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCAAdFiEEgOTj3suS2urGXVU3Y4mL3PG3PloFAl1/NNMACgkQY4mL3PG3
PlrTaBAAtSpZxpx+klDe7Ic8UnmaYCPlu2pp4JVx55Zo5INSHmJBFnIKn38NZkjo
KmnaQafSQVePFhyO93ZO6tLkb7rwsfvhwDVzkEHzvfLHlk5qeTDD57zH0nmhS/T2
MeXthJsKck3HP3BCA2v5LCEFXcjB1TiLRNG5v+/8897XbZPv7388xusHTMRjRKMW
111DVTfeiMFqFp2iFqXF+gcH0xEQy3dNv4cI+CBoHza2kD82r3jyt+K4KvhcdO79
VkWDX0yaQw0Hx99sZkP7DWywiAHHRS9tg7vDzPYZ5kYP1qqujkEqF0wAvc4mW8e3
sryAHrkX23PhBVqvhjIWHKxzbIVFzpTVxntbBIlPJo728e/7zOIVljqOH6V/PCRC
w48un16RmsxfHzNTSuFa3NKkDTBDfxhDRMPzWOED00+X3dZvgUTJQbtyk9DyA8wd
IDvsjcZ3bkZDtEZhJB7jwrMHd6QBeo3cvGXwQEjuJMAdDHXurrCpEaIDzLTHI71U
Rrk0I0pd6r0f5zyE/dA9pMVLF5e5w8idMk4smPVzpBBgFRy6TtAq/i/60IqvTeAf
cUs1bFimAJqc8Q+DQgLZvBEDO7R0JiD0v8I0EDtXaOJiwILQwWfqgNrpJCoRkRcp
gEg0WiwsG3XOe9zkyuXBm9yCYb3VixVhZaCugc5j1P5i/0m+Xmk=
=9ch+
-----END PGP SIGNATURE-----

--zzllgpaayqdv4sjx--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20190916070805.ykasrsmqdol3jwqt>