Re: [tin 2.4.3] snapshots - please test

Corinna Vinschen <[email protected]> Tue, 3 Jul 2018 17:55:30 +0200
Newsgroups gmane.network.tin.devel
Message-ID <[email protected]>
--===============8840513299203388063==
Content-Type: multipart/signed; micalg=pgp-sha256;
	protocol="application/pgp-signature"; boundary="GRPZ8SYKNexpdSJ7"
Content-Disposition: inline


--GRPZ8SYKNexpdSJ7
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Jul  3 17:13, Urs Jan=C3=9Fen wrote:
> In <[email protected]>, Corinna Vinschen wrote:
> > Just built on Cygwin.  Looks good.  Apart from a few "unused variable"
>=20
> Thanks, for the feedback, those are "normal" and depend on
> --enable-nntp or --enable-nntp-only.

I guessed so.

> > and "unorm_normalize_62 is deprecated" warnings,
>=20
> this is on the TODO list
>=20
> | o  add check for ICU unorm2.h/unorm2_normalize(); prefer it over
> |    ICU unorm.h/unorm_normalize()
>=20
> volunteers? ,-)

Hmm, the call has changed significantly.  Not that easy...

> > there's one which is ...interesting:
> >=20
> > pcre/pcre_study.c:419:35: warning: =E2=80=98<<=E2=80=99 in boolean cont=
ext, did you mean =E2=80=98<=E2=80=99 ? [-Wint-in-bool-context]
> >              if ((tcode[c/8] && (1 << (c&7))) !=3D 0)
> >                                 ~~~^~~~~~~~~
> >=20
> > The && looks wrong.  In the equivalent piece of code in upstream
> > pcre-8.42 this reads:
> >=20
> >              if ((map[c/8] & (1 << (c&7))) !=3D 0)
>=20
> the included pcre source is very outdated (> 10 years old now) and should
> either be updated or dropped (at least any system wide installed version
> should be preferred).

Dropping the included pcre would be preferrable, otherwise you get
outdated code by default.  It's even dangerous as fallback.  Better to
bail out from configure if the system has no libpcre installed.

>=20
> | o  clean up included pcre stuff and update to >=3D pcre-8.42
> |
> | o  and pcre2 support (new API)
> |    <http://www.pcre.org/current/doc/html/>
>=20
> But I'll fix that in the code. Thanks for the hint.
> =20
> > For the Cygwin release I will switch to --with-pcre for the time being.
>=20
> This is highly recommended.

Thanks,
Corinna

--=20
Corinna Vinschen
Cygwin Maintainer
Red Hat

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

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

iQIzBAEBCAAdFiEEoVYPmneWZnwT6kwF9TYGna5ET6AFAls7nHIACgkQ9TYGna5E
T6DS1Q//dhT1HozVt7MdPV2bGYdpxQsD7Bh0RxAgNRBTtBnaCEpAIE61U85S6dm2
Mq2mrWOD/89YEfl/m1NlKUfM1gUHKHe/b9TJvJCcoWXtDPIYOElv3I8h17p7p09Y
MUX6m2rwUqMsLAj3OO8GujWQCn8Zg7OdcrzNulL7pRK/KOV73VQxW8bQcbsLU1MN
DdG58ZrBlzN6J5xyoJO/kpaFJYhoV+l3e4vSl1zq8SF2JwPXqYzoIyTOZ6CoKghM
JpALWWex4rma9MOY7p4jKNtfx1JHFzeiGHkZ+70HKXmUHl6nGD6QMtuVHrEYLmL9
FO9Dr8qGvqYsFB5O3aMS7QO67/UniKwNByuZswOhS8V0N8kY2aYKbmL3sz7/gsII
+RdUdkMHLiVN+k6yGDfsQqO/0qS1UoHk8EVLwMqWprK7kzlLGD2cxR0jHWVCNpZD
nTraz+LqMNtlcsUO95JeyQ+hZs1zPfmp57aZ/5dAIK/EqRklUrJjWBXvMgzvMhI3
hUIH5JWdeatQ2oGwde7Zr9wHWWsSBfysCc5iT9xTm8IA2NgYa3TYlA4/uW5hp6OW
bkehNSdpFvJgXOnV4A+dEuEedIvUkdKM0HFv3pdJ9NQ13c3UHp/HAoIxlXVX4CkK
fW/T99pq73gMGWY1W5pQELjrYkWAbIxOb96C8q4ugkRyFngyILE=
=SDWJ
-----END PGP SIGNATURE-----

--GRPZ8SYKNexpdSJ7--


--===============8840513299203388063==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: base64
Content-Disposition: inline

X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KdGluLWRldiBt
YWlsaW5nIGxpc3QKdGluLWRldkB0aW4ub3JnCmh0dHA6Ly9saXN0cy50aW4ub3JnL2NnaS1iaW4v
bWFpbG1hbi9saXN0aW5mby90aW4tZGV2Cg==

--===============8840513299203388063==--