Peter M. Groen / libmodbus

Since SPDX release 3.0 LGPL-2.1+ became LGPL-2.1-or-later. Hence
replace the deprecated identifiers.

This replaces the lengthy license text headers with a short
and standardized license tag. See http://spdx.org for details.

This is useful e.g. for license compliance tools which scan
through files and generate a report of the licenses used
in a project.

Signed-off-by: Michael Heimpold <mhei@heimpold.de>

https://www.securecoding.cert.org/confluence/display/seccode/DCL37-C.+Do+not+declare+or+define+a+reserved+identifier
Not applied to libmodbus constants for now...

Thanks to oldfaber.

Signed-off-by: Michael Heimpold <mhei@heimpold.de>

Original patch for new connect function by Thomas Stalder.

- export new symbols to create specific behavior (RTU)
- the flag can't only be set by slaves
- new tests

Warning, modbus_receive returns 0 when a query is ignored.

- function removed from TCP and RTU code and from backend structure
- updated documentation
- updated examples

The function name was confusing because the write operation is performed
before the read. Take care to swap the arguments in the migration process.

Avoid incompatibility with GPLv2 program. This change has been approved by
Tobias Doerffel, Florian octo Forster and Hannu Vuolasaho.

The following functions have been renamed:
- modbus_get_timeout_begin -> modbus_get_response_timeout
- modbus_set_timeout_begin -> modbus_set_response_timeout
- modbus_get_timeout_end -> modbus_get_byte_timeout
- modbus_set_timeout_end -> modbus_set_byte_timeout

The meaning of these timeout intervals is now clearer. The documentation
has been updated.

- updated comments
- renamed internal constants (eg. _TIME_OUT_BEGIN_OF_TRAME in
  _TIME_OUT_BEGIN_OF_MESSAGE)

Reported by Allan Cornet.

The goal of this rewriting is to avoid the timeouts on the receiving
of exceptions and to be more robust on bad requests. Some devices
use the exception MODBUS_EXCEPTION_ACKNOWLEDGE to response to some
valid requests, so in this case, you'll really appreciate this
change!

- Slower! More system calls are used.
- The code is cleaner and easier to understand.
- Really faster when an exception occurs.
- Fix unit test of bad request in RTU

Reported by Ivan Giuliani <giuliani.v@gmail.com>
It was not possible to use libmodbus outside of projects without
config.h file.

The inclusion has been removed from the public header.

Signed-off-by: Stéphane Raimbault <stephane.raimbault@gmail.com>

These functions have no meaning in RTU so it's better to specialize
the names and remove them from the backend.

- remove the functions from the backend
- update tests to handle RTU mode (master and slave)
- add command line options to tests (rtu or tcp)

- check strlcpy in configure.ac
- test for empty device string
- test truncated device string

This change fixes a compilation problem in MinGW. As general rule,
each header must be as independent as possible.

Introduced the select() operation for backends as the WAIT_DATA macro
approach is not very extensible and causes trouble when adding platform-
specific codepaths.

Signed-off-by: Stéphane Raimbault <stephane.raimbault@gmail.com>

The send() and recv() functions of the backends might require more
information than just a file descriptor, therefore pass the complete
modbus_t structure.

Signed-off-by: Stéphane Raimbault <stephane.raimbault@gmail.com>

Renamed error_print() to _error_print() in order to indicate it's a
private method and make it globally accessible so functions in
modbus-rtu.c and modbus-tcp.c can make use of it as well.

Signed-off-by: Stéphane Raimbault <stephane.raimbault@gmail.com>

All private functions and constants are now prefixed by _.
The modbus protocol uses entry points to call specific functions.

- more coherent
- namespace
- opaque and smaller context
- usual wording