[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: bison-20001221

From: Akim Demaille
Subject: Re: bison-20001221
Date: 18 Jan 2001 12:24:16 +0100
User-agent: Gnus/5.0808 (Gnus v5.8.8) XEmacs/21.1 (Crater Lake)

>>>>> "Hans" == Hans Aberg <address@hidden> writes:

Hans> At 14:17 +0100 1-01-12, Akim Demaille wrote:
>>> - On a number of places there is include <config.h> It should be
>>> include "config.h"
>> I never quite understood the difference between the two, and some
>> people will tell you to do just the opposite (see the Autoconf doc
>> for a start).

Hans> #include <...> searches on the library directories, whereas
Hans> #include "..." first searches the local directories and then the
Hans> library ones -- or so I think is the normal way.

I of course meant in the context of Autoconf, but end of the debate,
or move to address@hidden if you wish, this is not the right place.

Hans> Another idea I had is to make obstack into a FILE

May I recall that we moved to obstack to avoid temp files?

Hans> - I got around the problem by changing the obstack_fgrow1 macro
Hans> containing a static array: static char buf[4096]; Not pure
Hans> anymore, but it does not make any difference in my port.

Bleah.  What size would make you happy?  I've used 4096 knowing it's
way too much.

Hans> Unnecessary if "quotearg.h" includes <size_t.h>.
>> ???  No it does not.

This is the famour `size_t.h' I was referring to in another mail, and
you told me there is no such thing.  So what is the problem then?

Hans> The guys writing those libraries you stole seems to have a
Hans> penchant for #include <..> instead of #include "...". This means
Hans> that when you steal them and pack them in a distribution which
Hans> is not going to be installed where the standard <...> headers
Hans> are, one needs to change them to "...". :-)

We disagree on the understanding of <..> vs "..", but that debate is
not related to Bison, so let drop it.

Hans> - File "quote_arg.c", "else" clause of "#if HAVE_MBRTOWC": macro
Hans> "iswprint" is already in the C-standard, so redefining it here
Hans> causes an error.
>> This is good to know.  Please let me see the error.

Hans> I don't know what you mean here.

What does the compiler say etc.  And BTW, if you want to report this,
do it to Paul Eggert <address@hidden>, not me, as I don't maintain
quotearg.  Bison is no longer concerned, let's drop this issue too.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]