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.

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.