warning in generated code, with -Ca

warning: conversion to 'yy_state_type' from 'flex_uint32_t' may change the sign of the result [-Wsign-conversion]

suppress `WARNINGFLAGS' for the almost obsolete `noansi' test cases

configure option `--enable-warnings' and `WARNINGFLAGS'

`WARNINGFLAGS' can be passed when invoking `configure'
 and when invoking `make'

if configure switch `--enable-warnings' was given
  then default to something useful if we have `GCC'

`WARNINGFLAGS' is not used when compiling `stage1flex'
  to avoid unnecessary clutter

Fix potential buffer overflow in strncat()

When using clang/llvm 3.8 to compile flex, the following warning is
emitted:

main.c:378:27: warning: the value of the size argument in 'strncat' is too large, might lead to a buffer overflow [-Wstrncat-size]
                                        strncat(m4_path, m4, sizeof(m4_path));
                                                             ^~~~~~~~~~~~~~~
main.c:378:27: note: change the argument to be the free space in the destination buffer minus the terminating null byte
                                        strncat(m4_path, m4, sizeof(m4_path));
                                                             ^~~~~~~~~~~~~~~
                                                             sizeof(m4_path) - strlen(m4_path) - 1

Fix it up by using the solution proposed by the warning message.

build: simplified dependency tracking so parallel make runs succeed

avoid warning in generated code, with -Cf

warning: conversion to 'unsigned int' from 'int' may change the sign of the result [-Wsign-conversion]

mention v2.6.2; summarize changes since 2.6.1

doc: corrected example in manual, gh#67

warning: conversion to 'size_t' from 'int' may change the sign of the result [-Wsign-conversion]

warning: conversion to 'long unsigned int' from 'int' may change the sign of the result [-Wsign-conversion]

warning: conversion to 'flex_uint32_t' from 'long unsigned int' may alter its value [-Wconversion]

struct yytbl_reader, member bread is of type flex_uint32_t

warning: redundant redeclaration of '...' [-Wredundant-decls]

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

warning: unused variable 'fp' [-Wunused-variable]

main(), warning: redundant redeclaration of 'main' [-Wredundant-decls]

main(), warning: old-style function definition [-Wold-style-definition]

main(), warning: old-style function definition [-Wold-style-definition]

Fix two more casts

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