Fixes yyl compare with unsigned warning

Suppress portability warnings in Makefile generation

generated code, in yyensure_buffer_stack(), change type of local `num_to_alloc'

which is exclusively used in yy_size_t context

generated code, in yy_get_next_buffer(), change type of local `number_to_move'

suits better, because `yy_n_chars' and `yy_buf_size' are of type `int'

generated code, `_yybytes_len' is of type `int', fix code accordingly

generated code, `max_size' seems to be of type `int', fix casts accordingly

generated code, here `new_size' is of type `int', fix casts accordingly

generated code, `yy_buf_size' is of type `int', fix casts accordingly

generated code, `offset' is of type `int'

generated code, `yy_more_len' is of type `int'

scan.l, rewrite two loops to avoid unneccesairy casting

improve readability

another cast in tblcmp.c to avoid warning

casts in buf_append() to get rid of warnings

cast to suite type of flex_uint32_t td_lolen

cast to get rid of warnings

cast and fix usage of log10(), ceil to prevent buffer overflow

tables.c, sprinkle casts to get rid of warnings

yytbl_data_compress(), change type of local newsz to get rid of warnings

change type of struct yytbl_writer.total_written to get rid of warnings

change argument type of yytbl_writen() to get rid of warnings

yytbl_write8/16/32(), change type of local variables to get rid of warnings

change return type and rename int htoi()/otoi() --> unsigned int htoui()/otoui()

Mention 2.6.1 release date

avoid warning, add (int) cast to the read() return value

For similiarity with the fread() case.

avoid warning, POSIX says yyless() has an `int' argument

use type size_t in filter_tee_header() to avoid warnings

add (size_t) casts to malloc invocations to prevent warnings

add (int) casts to some strlen() invocations to prevent warnings

ndlookup(), char *, to get rid of casts and warnings

ndinstal(), char *, to get rid of casts and warnings

cclinstal() and ccllookup(), char *, to get rid of casts and warnings

warning: redundant redeclaration of ‘gen_next_state’ [-Wredundant-decls]

warning: no previous prototype for ‘mkecstbl’ [-Wmissing-prototypes]

warning: suggest parentheses around assignment used as truth value [-Wparentheses]

warning: redundant redeclaration of ‘yywrap’ [-Wredundant-decls]

warning: redundant redeclaration of ‘yylval’ [-Wredundant-decls]

warning: redundant redeclaration of ‘yyin’ [-Wredundant-decls]

warning: redundant redeclaration of ‘yyparse’ [-Wredundant-decls]

unification, rename some more

rename these too for improved similiarity:
  OPTION_OP OPT_HEADER OPT_EXTRA_TYPE OPT_TABLES

fix name clash, OPT_OUTFILE from parse.y and from options.h

these collide:
  OPT_OUTFILE OPT_PREFIX OPT_YYCLASS
rename them TOK_... in the parser

Prototyped reallocarray implementation

Removed custom strcasecmp() function

fwrite wants a size_t, yyleng is int per posix

Remove some unneeded casts

warning: negative integer implicitly converted to unsigned type [-Wsign-conversion]

dfa.c:157:24: warning: conversion to 'size_t' from 'int' may change the sign of the result [-Wsign-conversion]

most certainly safe cast

ccl.c:86:19: warning: conversion to 'unsigned char' from 'int' may alter its value [-Wconversion]

ch seems to have been checked for proper range some lines above

warning: deprecated directive, use '%pure-parser' [-Wdeprecated]

warning: no previous prototype for 'do_nothing' [-Wmissing-prototypes]

included stdlib.h header

Described more post v2.6.0 changes

build: Removed bzip2 distribiution archive

removed obsolete program check

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.