Ethereal-dev: Re: [Ethereal-dev] 0.9.13 patches

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

From: Guy Harris <gharris@xxxxxxxxx>
Date: Thu, 12 Jun 2003 19:47:05 -0700
On Thu, Jun 12, 2003 at 06:07:57PM -0500, Albert Chin wrote:
> 2. wiretap/ascend-scanner.c
>    "config.h" should always be the first include file

Unfortunately, CVS wasn't very happy when I tried to check in the
results of that patch:

	% cvs commit ascend-scanner.c
	cvs commit: use `cvs add' to create an entry for ascend-scanner.c
	cvs [commit aborted]: correct above errors first!

Unfortunately, you can't move the include of "config.h" earlier in the
source from which "ascend-scanner.c" is generated, namely
"ascend-scanner.l", because the first include in "ascend-scanner.l"
already *is*

	#ifdef HAVE_CONFIG_H
	#include "config.h"
	#endif

and Flex insists on putting other includes before the first include in
the Flex file, so you'd either have to somehow convince Flex to put some
text *before* all the generated code, or find some way not to make it a
requirement, on whatever platform it's a problem, to have "config.h" be
first.