Thomas Dickey wrote:
On Mon, 2 Jul 2007, Mikhail T. wrote:
Hello!
This is a minor patch, which makes sure, the correct entities.h is
included. The file-name is common, and, depending on what else is
installed on the building machine, a wrong file may be included -- as
just happened on my machine. The patch makes the include more explicit:
I thought I addressed that in dev.5 (with the include-ordering).
Dunno about the dev.5 -- I was trying to build 2.8.6rel.5 and the build
was failing due to entities.h installed by gd-2. Include reordering is a
good idea, but using <> for your own headers is wrong, IMO. My patch
offers a "bullet-proof" way of ensuring, the right header is included in
this case.
Yours,
-mi
--- src/LYCharSets.c Sun Dec 18 17:52:30 2005
+++ src/LYCharSets.c Mon Jul 2 16:10:40 2007
@@ -871,5 +871,5 @@
UCode_t HTMLGetEntityUCValue(const char *name)
{
-#include <entities.h>
+#include "chrtrans/entities.h"
UCode_t value = 0;
Please, include it in the next (pre-)release. Thank you,
-mi
_______________________________________________
Lynx-dev mailing list
[email protected]
http://lists.nongnu.org/mailman/listinfo/lynx-dev
