Made some program checks more robust

Remove unneeded cast to int

Fixed incorrect integer type

Fix more integer types, resolves sf 184, 187

Removed some type conversion warnings

Changed another buffer size to int; resolves gh#61

Changed type of yy_n_chars to int; gh#53, sf#160.

The variable yy_n_chars had been of type yy_size_t which is incorrect
given its use in read(). While it might be adviseable to look at
defining a yy_ssize_t, there might be some issues doing this and so, for
now, at least, we'll punt back to int.

Fixed size of bufferallocation, resolved gh#54.

The value of n_alloc was a count, not a size. Multiplying the value by the element size was incorrect. That multiplication was already being done and having it done twice was incorrect.

Allow '%option noline' in flex input file, resolves gh#56.

Allow specifying '%option noline' in the input file, leading to the same
effect as calling flex with the command line option --noline.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

Emit no #line directives if gen_line_dirs is false, resolves igh#55.

There are two instances in the code which will print a #line directive
to the resulting lexer, regardless of the value of gen_line_dirs. Fix
them, so they also respect gen_line_dirs.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>

Converted K&R style function definitions to ANSI C style

Consistently make use of the ANSI C function definition style instead of
the K&R style.

Used NULL constant instead of plain integer for NULL pointer.

The sparse static checker warns about using plain integer 0 as NULL
pointers in the generated lexer code. Fix this by using NULL
consistently for pointers.

Marked declaration and definition of yy_fatal_error as noreturn.

Only the declaration of yy_fatal_error is marked with
__attribute__((__noreturn__)) in case GCC >= 3 is used, but not the
definition. This leads to the sparse static checker to complain about
function declaration mismatch.

Fix it by defining a macro yynoreturn and using it for both the
declaration and the definition of yy_fatal_error.

Fixed declaration mismatch in yy_fatal_error.

The prototype declares yy_fatal_error parameter as "const char msg[]"
while the definition uses "const char* msg" (introduced by commit
e9d5fc713f61b) which causes the sparse static checkers to produce an
error.

Fix this by adjusting the definition to use "const char* msg" as well.
Also change the C++ version accordingly so it matches the declaration in
FlexLexer.hpp.

flex is for flex #NCoC

Referred to github for issue tracking, no longer sf

Opened files in binary mode explicitly

Linked flex binary against libintl, not libfl.

Signed-off-by: Thomas Klausner <wiz@NetBSD.org>

Improved pipe-stdin hack behavior; resolves sf#198.

Signed-off-by: Thomas <Klausner wiz@NetBSD.org>

Removed no longer needed header checks

Checked for reallocarray() with AC_REPLACE_FUNCS

include libgen.h from flexdef.h, not main.c

Replace basename2() with basename(3).

Given the following program:

\#include <libgen.h>
\#include <stdio.h>

/* extracts basename from path, optionally stripping the extension "\.*"
 * (same concept as /bin/sh `basename`, but different handling of extension). */
static char *basename2 (char *path)
{
char   *b;

for (b = path; *path; path++)
if (*path == '/')
b = path + 1;
return b;
}

static void basename_compare(char *path)
{
printf("basename: %s\n", basename(path));
printf("basename2: %s\n\n", basename2(path));
}

int main (int argc, char *argv[])
{
// From http://pubs.opengroup.org/onlinepubs/9699919799/
// ``Sample Input and Output Strings''
basename_compare("/usr/lib");
basename_compare("/usr/");
basename_compare("/");
basename_compare("///");
basename_compare("//usr//lib//");
return 0;
}

... and the program's output:

basename: lib
basename2: lib

basename: usr
basename2:

basename: /
basename2:

basename: /
basename2:

basename: lib
basename2:

... we can see that basename2() behaves the same as basename(3) in the
average use case, but messes up pretty severely in others.  Besides
that, basename(3) is mandated by POSIX so should be present on modern
Unix-like systems, so we shouldn't define it ourselves.

Some notes:
- it doesn't appear to be mentioned in POSIX, but OpenBSD's basename(3)
  returns NULL if the returned path componenet is > PATH_MAX, so add a
  check for that
- basename(3) shouldn't return an empty string, so remove the
  program_name[0] != '\0' check

Simplify basename2().

It's only call site does not activate the `strip_ext` code path, so the
function can be simplified a lot.  While here, remove a double
assignment.

Cleaned up __STDC__ #ifdefs.

Assuming a compiler conforming to the ISO C standard is used, i.e.,
__STDC__ is defined to 1, YY_USE_CONST is always defined and can be
eliminated.

Remove remaining use of PROTO

buf.c: use snprintf

build: reformatted AC_CHECK_FUNCS for readability

correct function prototype

Remove more instances of PROTO

Removed prototype for main().

It's not called anywhere else so the prototype is not needed.
See the C99 standard [1], section 5.1.2.2.1 for more info.

[1]: http://www.open-std.org/jtc1/sc22/wg14/www/docs/n1256.pdf

Removed more instances of PROTO, ansifying.

Noted github for issue tracking and pull requests

Reformatted README; removed sf bug tracking link.

Removed macros for indentation level.

Just increment or decrement the indentation counter. That's less to
remember and makes the code more readable.

w# Please enter the commit message for your changes. Lines starting

tests: fixed paths to input files.

The current test wrapper works only when the inputs are specified using
relative paths.  If they're specified with absolute paths, the driver
fails to detect the inputs because it always prepends the input dir name
which itself is a relative path:
$ cd tests
$ ./testwrapper.sh -d . -i $PWD/reject.txt -t ./reject_ver.table
<fails to open inputs>

This normally doesn't show up because people run `./configure` or, for
out of tree builds, `../configure`.  But if you happen to run configure
with an absolute path, then automake tends to generate absolute paths
as well leading to test failures.

Fix all of this by dropping the implicit input directory prepending.

- INPUT_NAME is often a list of files, not just a single one
- the input directory is used to find the testname tables which are
  usually generated, so it's impossible to use files from both source
  and build directories
- most of the time, the full/correct path is already specified

configure: fixed realloc test.

The [] characters are used for quoting in m4, so the attempt to use
them in place of `test` fails yielding the warning at build time:

.../flex/configure: line 20222: no: command not found

Started removal of PROTO macro.

The PROTO macro is no longer needed. Additionally, its usage is inconsistent,
so we began removing it.

Removed SHORT_FILE_NAMES preprocessor symbol.

As a relic of MS-DOS, we don't need this.

It's never defined; see 13b5b214f53d1c3354a7ab910bd160c126df1331.

Removed additional MSDOS ifdef.

Added new function reallocarray.

This is taken from OpenSSH Portable, which in turn takes it from
OpenBSD.

reallocarray wraps the stdlib's realloc function. It takes two size
arguments and checks for overflow, like calloc, but doesn't zero the
memory. Therefore, it allows us to do overflow-safe array reallocations
and overflow-safe unzeroed array allocations, which the stdlib
allocation functions don't.

We have a bunch of specific array allocation macros, none of
which check for overflow. reallocarray should be able to replace them.

Removed MS-DOS, VMS macros.

Given the age of the MS-DOS and VMS platforms, it's likely that no one is building flex on them any more. Additionally, the preferred approach is to test for particular platform features rather than to test for particular platforms.

Made search for m4 more explicit.

Returned 0 from yywrap() instead of EOF

Changed end of main() in libmain to exit(0)

Mentioned v2.6.1; documented some changes since v2.6.0

Updated build documentation; finished sf#155.

Removed version numbers for build tools. Noted that version requirements for build tools will be noted in configure.ac. Expanded documentation of building texinfo based docs.

Removed flex.pdf from distribution; partially fixed sf#155

Commented in C style in skeleton; fixed sf#195

Ignored autoscan files

Checked for ranlib and strdup() at configure time

Replace zero_out() with stdlib's memset.

Removed memory allocation casts.

Removed alloca() configuration.

Since alloca() is dangerous, depricated, we remove it.

Removed implementation of isascii().

POSIX defines isascii(), so it's likely present on anything weactually build flex on these days.

Removed NULL-checks before free()

Removed flex_free()i, corrected buf_destroy logic.

As with flex_alloc(), replace with direct calls to free().

The function buf_destroy is now null safe and the logic was corrected to free()
correctly.

Removed flex_realloc().

As with flex_alloc(), replace calls to flex_realloc(), which was just a
wrapper around realloc().

Removed flex_alloc; cleaned up style.

The function flex_alloc() was just a wrapper around malloc(). Since this only added unclarity, and the flex_alloc() function is likely a legacy of olden times, remove it in favor of calls to malloc() directly.

Style elements cleaned up:

 * superfluous spacing around parentheses
 * non-constant initialization in variable declarations
 * needless casts
 * almost all uses of assignments as subexpressions

Remove allocation casts

Built flex with itself.

Changes in scan.l need to be built into flex with the same version of
flex in some cases. Since this build requirement is minimal, we simply
bootstrap flex unconditionally.

We intentionally exclude from version control the bootstrap artifacts as
the extra copy of the lexer, the intermediate scanner and the bootstrap
executable are not of interest.

Use NULL rather than (type *) 0.

Replace copy_unsigned_string() with xstrdup().

Like copy_string(), copy_unsigned_string() is just a clone of the
stlib's strdup(). We only use it twice. I'm pretty confident that char
signedness is irrelevant in this case.

Cleaned up white space.

Removed struct keyword before yy_buffer_state

Replaced FLEX_STD macro with std::.

The std:: construct exists as of C++98, so we can simply assume it is
supported.

Generated skel.c explicitly in srcdir.

Rewrote the target for skel.c to explicitly mention the srcdir. This
should help when building flex from a directory outside the flex
tree. Spread the rule out over several lines to enhance readability.

Called glibtoolize if libtoolize run fails.

On Mac OS X, libtoolize is known as glibtoolize. In cases where libtoolize is not present, then calling glibtoolize when bootstrapping the build system gives more folks a shot at getting flex built from the ground up.

Checked for (g)texi2dvi. Better bison, help2man checks.

Added test for presence of (g)texi2dvi program. Gave notice if texi2dvi
is unavailable and set TEXI2DVI=: to avoid giving users headaches.

Enhanced tests for bison and help2man with notices when the programs
aren't found.  Set their program variables to use the missing script
in build-aux since it's compatible with them.

Made string copying more standard.

copy_string() was a clone of the stdlib's strdup(). For safety,
simplicity, and speed, we should use that instead. We introduce xstrdup() which wraps strdup() in a failure upon memory allocation errors.

Error on unbalanced parentheses in rules section.

Cleaned up BUILT_SOURCES list.

Removed reference to skel.c as a built source since other make rules cover this case.

Sorted file names in flex_SOURCES

Replaced CHAR macro with unsigned char type.

Thanks to Michael McConville for pointing out that the old Char macro
causes problems with static analysis.  The macro has been removed and
replaced with 'unsigned char' throughout the flex sources. The macro is
not needed at best and was confusing at worst. It was not used in any of
the example files nor was it mentioned in the manual at all.

updated syntax of AC_INIT call as per autoupdate

removed extra call to a _CPPFLAGS variable

Switch function definitions from mixed K&R to consistent ANSI C.

flex was using K&R function definitions for some functions and
ANSI C style in others, sometimes even in the same file. Change
the code to consistently use ANSI C.

Signed-off-by: Stefan Reinauer <stefan.reinauer@coreboot.org>

Used in-tree header file for c++ tests.

Declared version 2.6.0

Dated, described flex release 2.6.0

Declared version 2.6.0rc1

Supplied versioning information in flex libraries.

Resolves sourceforge bug #182. On OSX, and possibly other platforms, building the libfl libraries without versioning information caused a build failures.

Cleaned up more precisely after make check.

BUILT_SOURCES is now just the list of headers built as per the
automake manual. We provide the list of files to clean to make
rebuilding the test suite programs easier. We then use the CLEANFILES
list in a dist-hook to clean up the distribution that automake gathers
since not distributing flex generated files is foreign to automake's
mindset, but we need exactly that.

Additionally, we locate inputs to the tables-related tests more
precisely. Some files are in srcdir and some are in builddir, which
the arguments to the log compiler are now made aware of.

Pulled out scripts in tests/Makefile.am

Added srcdir to tableopts.am include

Changed man page dependencies.

The man page is just the --help output as reformatted by help2man. The
--help option is most likely to change when the flex skeleton changes
or one of the option parsing files changes or the configure.ac script
itself changes. The dependencies reflect this now.

It is still necessary, under some circumstances, to rebuild flex
explicitly before building the man page. In theory, it's possible to
have automake arrange to do this all the time, but doing so works out
to be fragile, given the rest of the build system.

Called libtoolize directly because autoreconf fails to pick up LT_INIT properly

Added tableopts.sh to EXTRA_DIST

Made tests depend on the built flex binary.

Merge branch 'master' into streamrefs

Cleaned more generated sources.

Added skel.c explicitly to MAINTAINERCLEANFILES in src/Makefile.am.

Added a bunch of files to built_SOURCES in tests/Makefile.am so that the
suite is easier to clean up.

Updated documentation to reflect the revisions to FlexLexer.h

Changed several pointers to istream (and ostream) to references in c++-only sections of the skeleton.

Patched up a variety of expected errors caused by changing istream* to istream&.

Added a stray 'make' at line 545.  Oops.

Changed the buffer_state struct to store std::streambuf* instead of std::istream* for C++ mode.  Changed interfaces in FlexLexer.h to take std::istream& instead of *.  Backward compatibility temporarily broken.

Patched up backward compatibility with reasonable behavior in the presence of null pointers.

Re-added backward-compatible versions of the yyFlexLexer methods that take iostream pointers.  All tests passing.

new zh_CN translation from the translation project

fix possible resource leak with yynultrans_tbl

fix possible uninitialized array values

add %{...%} block to example in manual

initialize libtool earlier in build system generation

new da translation from the Translation Project

Fix `label unused` warning

Fix two "signed/unsigned" warnings