CVS log for src/tests/lib/libcurses/slave/slave.c

Up to [cvs.NetBSD.org] / src / tests / lib / libcurses / slave

Request diff between arbitrary revisions

Keyword substitution: kv
Default branch: MAIN

tests/libcurses: add parentheses around argument to sizeof

tests/libcurses: extract read_command_argument from process_commands

Having code indented so far to the right that each word gets its own
line is ridiculous.  Fix that.

While here, remove the cargo-cult realloc pattern, which is not needed
if the process exits immediately on error.

While here, reduce the indentation of the code by returning early.

No functional change.

tests/libcurses: fix reading from the parent process

In case of a short read, processing the incomplete data invoked
undefined behavior.

tests/libcurses: fix child process handling

The child process needs to be properly controlled by the parent process.
Otherwise it is not possible to get code coverage data from it using
gcov since that requires the child process to exit normally.  Previously
the child process had been killed because its parent had exited.

tests/libcurses: clean up remaining include guard, comments

No functional change.

tests/libcurses: fix pipe handling in child process

The child process only ever needs 2 ends of the pipes: one for reading
the commands, one for writing back the results.

tests/libcurses: format misleading do-while loop

tests/libcurses: add missing space after control flow keywords

tests/libcurses: fix typo in license text

Merge in code from Google Summer of Code project which dramatically
increases the number of tests performed.

Thanks to Naman Jain <jnaman806@gmail.com> for his excellent work on
this GSoC project.

Mostly merge changes from HEAD upto 20200411

Rototill to add complex character support:

- Add cchar command to director test language to allow a complex character
  to be defined and passed to the slave.
- Removed the distinction between args and returns internally to prevent
  confusing the two which leads to errors and/or crashes.
- Converted mutt_test to use the new cchar definition and fixed the
  check files for the test.

- add support for getparyx and getyx calls
- allow a NULL pointer to be returned to the director
- add support for passing back a single chtype character
- fix some indentation
- fix a lot of curses command calls that were just plain wrong
- don't try to allocate storage for a NULL parameter, it doesn't need it

Catchup with rmind-uvmplock merge.

- pass things in arguments instead of environment
- sanity check arguments
- disable extra data warning, old tests gave it too
- print more detailed errors
- use err/warn more
*some tests still fail; more than they used to*

Turn warns on for all tests and fix all the bugs.

add missing header files.

Allow a NULL pointer to be passed as an argument to a call.

Add atf for libcurses with some tests