windows: Always check and set ReadTotalTimeoutMultiplier.

This is not currently touched, but will be by the upcoming
sp_blocking_read_next() function.

posix: Make sure blocking operations can't time out without trying.

posix: Consistent debug output when blocking operations time out.

posix: Move FD_ZERO() and FD_SET() calls outside loops.

windows: Always check return value of GetOverlappedResult().

windows: Restructure if/else blocks with unnecessary indentation.

windows: restart_wait_if_needed(): Inline receive buffer check.

This removes all the debug spam arising from calling sp_input_waiting().

windows: restart_wait_if_needed(): Skip buffer check if no bytes read.

windows: Refactor duplicated code into a static function.

windows: Avoid unnecessary calls to SetCommTimeouts().

This fixes bug #586.

windows: Strip CR/LF from end of system error messages.

This fixes bug #585.

doc: correct structure name in sp_get_port_handle() documentation.

libserialport.h.in: Fix/update some API docs.

sp_list_ports(): Actually set list_ptr NULL as documented.

Fix a potential segfault in sp_get_config_*().

Fix a potential segfault in sp_get_port_handle().

windows.c: Constify a few function parameters.

sp_get_port_usb_vid_pid(): Document that usb_vid/usb_pid can be NULL.

Rename timeout to timeout_ms for clarity in a few places.

autotools: Don't set/override "user variables".

This fixes bug #577.

Makefile.am: Simplify LDFLAGS handling a bit.

Constify the sp_get_port_description() parameter.

Constify the sp_get_port_transport() parameter.

configure.ac: Drop no longer needed SP_PKGLIBS.

Various Doxygen comment updates and fixes.

 - Add some missing @param lines.

 - Add [in], [out], or [in,out] markers for @param tags.

 - Add explicit "Must not be NULL" comments for parameters where
   the caller should not pass in a NULL.

libserialport.h.in: Drop unneeded <windows.h> #include.

This is not needed and would pollute the namespace of every
libserialport user on Windows.

libserialport.h.in: Add @since tags for new enums.

Set package version to 0.1.1, use "@since 0.1.1" Doxygen tags.

The next release will not contain any backwards-incompatible
API or ABI changes, thus changing the package version to 0.1.1.

windows: Drop no longer needed -luuid.

Also, use SP_LIBS in Makefile.am.

This is build-tested using a native MinGW-w64 based toolchain
as well as a MinGW-w64 based cross-compile setup.

configure.ac: Minor whitespace cosmetics.

Doxyfile: Update to Doxygen 1.8.8 template.

(by running 'doxygen -u')

configure.ac: Add #include guard to config.h.

Fix a compiler warning when -Wshadow is used.

    CC       serialport.lo
  In file included from ../serialport.c:25:0:
  ../serialport.c: In function 'get_config':
  ../libserialport_internal.h:227:25: warning: declaration of 'ret' shadows a previous local [-Wshadow]
   #define TRY(x) do { int ret = x; if (ret != SP_OK) RETURN_CODEVAL(ret); } while (0)
                           ^
  ../serialport.c:1566:3: note: in expansion of macro 'TRY'
     TRY(get_baudrate(port->fd, &config->baudrate));
     ^
  ../serialport.c:1543:6: warning: shadowed declaration is here [-Wshadow]
    int ret = get_flow(port->fd, data);
        ^

Also, add -Wshadow to the list of default compiler options.

autogen.sh: Drop obsolete MinGW/MSYS items.

For Windows builds (which require MinGW-w64) we currently support:

 - cross-builds using MXE (mxe.cc) and possibly other cross-compile setups

 - native builds using MSYS2 (sf.net/projects/msys2/)

Neither of those require explicitly specifying ACLOCAL_DIR.

README: Refer to MSYS2 only.

The "old" MSYS related to the "old" and unsupported MinGW (from
mingw.org) may or may not work properly, so only refer to MSYS2 which
we currently recommend and which has been tested.

Minor Doxygen improvements and fixes.

configure.ac: Use $host_os instead of $target_os.

$build and $build_os refer to the machine on which the software is
built, $host and $host_os refer to the machine for which the software
is (cross-)built.

The variables $target and $target_os (despite the confusing names)
are only relevant in the context of building cross-compilers.

README: Add link to online API docs.

README: Mention FreeBSD port enumeration support.

configure.ac: Add -Wmissing-prototypes.

Fix two compiler warnings on platforms without enumeration support.

  serialport.c: In function 'sp_list_ports':
  serialport.c:326:6: warning: unused variable 'ret' [-Wunused-variable]
    int ret;
        ^
  serialport.c:325:19: warning: unused variable 'list' [-Wunused-variable]
    struct sp_port **list;
                     ^

add_handle(): Fix a realloc() related issue.

Fix a compiler warning.

This fixes the following scan-build warning:

  serialport.c: In function 'sp_list_ports':
  serialport.c:354:1: warning: control reaches end of non-void function [-Wreturn-type]

Fix indentation of a switch().

add_handle(): Fix a memory leak.

This fixes the following scan-build warning:

  serialport.c:1170:3: warning: Potential leak of memory pointed to by 'new_handles'
                  RETURN_ERROR(SP_ERR_MEM, "Mask array realloc() failed");
                  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Use libtool's -no-undefined option on all platforms.

Clarify sp_list_ports() code.

This also fixes the following scan-build warning:

  serialport.c:335:15: warning: Result of 'malloc' is converted to a
  pointer of type 'struct sp_port *', which is incompatible with sizeof
  operand type 'struct sp_port **'
          if (!(list = malloc(sizeof(struct sp_port **))))
                       ^~~~~~ ~~~~~~~~~~~~~~~~~~~~~~~~~

windows: Set fAbortOnError=FALSE.

Since we're not handling and clearing receive errors (framing, parity and
overrun), we should turn them off to avoid crashing out with
ERROR_OPERATION_ABORTED in various calls if they occur. Invalid data
will then simply not end up in the buffer. This is consistent with our
current behaviour on posix/termios systems.

It might be nice to be able to inform calling code about receive errors, but I
can't see a good way to do this in a cross-platform way at the moment.

This fixes (parts of) bug #341.

windows: Revise management of WaitCommEvent() operations.

This fixes (at least parts of) bug #341.

Make std_baudrates[] static (only used in one file).

Various cosmetic and consistency fixes.

libserialport_internal.h: Add #include guard.

Fix some typos.

linux_termios.h: Add missing #include of "config.h".

Generate a config.h rather than passing defines on command line.

Fixes bug #553.

Fix misleading snprintf() arguments.

If present, add usb_serial to description for better identification

Handle enumeration of native UART ports on FreeBSD

Implement FreeBSD OS-specific serial port detection and query

windows: Fix restart of RX/error wait operation after reads.

This fixes bug #421.

Fix free() of uninitialized pointer.

Explicitly set port->usb_path to NULL, otherwise a free(port->usb_path)
can happen where port->usb_path is uninitialized.

This closes bug #512.

windows: Properly handle return value of SetupDiOpenDevRegKey().

This fixes bug #499.

linux: Check for BOTHER macro at configure time.

This fixes bug #363.

Flush only the requested buffers

Fix the sp_flush() so that it flushes only the requested buffers
also in unix.

Fix typos.

windows: Fix compile issues with multiply or not defined GUIDs.

This fixes bug #416.

windows: Indentation (cosmetics only).

windows: Fix various memory leaks.

This fixes bug #419.

Add SP_MODE_READ_WRITE enum value.

README: Add URL to the libserialport website.

README: Mention that only MinGW-w64 works (but not MinGW).

windows: -no-undefined is required to make a DLL.

windows: Add -lsetupapi when building for WIN32.

windows: Include proper GUID defining header to fix build.

The initguid.h header must appear early or it does not have
an effect, at least on cross-build from Linux.

linux: remove readlinkat() dependency

linux_termios: fix return types to actually match returned values

This fixes the following warning :

linux_termios.c: In function 'get_termios_get_ioctl':
linux_termios.c:42:2: warning: overflow in implicit constant conversion [-Woverflow]
  return TCGETS2;
  ^

This fixes bug #396.

Fix Mac OS X build (untested).

Fix Windows build.

linux: Fix warning by marking vid, pid variables unsigned.

Compile with -pedantic.

linux: Define feature macros to get required functions.

This is needed to build with -std=c99.

Revise debug macros to work in strict C99.

Specialise RETURN_VALUE macro into RETURN_{INT,STRING,POINTER}.

This avoids the need to pass the required format string on every
call and also eliminates the need for the non-standard typeof() call.

libserialport.h.in: remove trailing commas in enum definitions

This makes it easier to use libserialport on projects that build with
the GCC -pedantic flag enabled.

Bump package version to 0.2.0.

Rename sp_get_signals() parameter to avoid name clash with Qt headers.

configure.ac: We no longer need to set WINVER explicitly.

Since we now require MinGW-w64, we no longer have to set WINVER manually,
since MinGW-w64 defaults to WINVER=0x0502 (which corresponds to
Windows Server 2003 SP1 / Windows XP SP2 apparently, which is fine for us).

windows.c: Remove MinGW workarounds, require MinGW-w64 instead.

We're switching from MinGW (i686-pc-mingw32.static in MXE)...

  http://mingw.org/

...to MinGW-w64 (i686-w64-mingw32.static in MXE):

  http://mingw-w64.sourceforge.net/

The latter is a fork which nowadays seems to be more active and also to
have a more complete set of win32 API headers and libs.

History:
http://sourceforge.net/p/mingw-w64/wiki2/History/

This closes bug #393.

windows.c: Define a bunch of stuff that's not in MinGW, yet.

Without these defineѕ the build fails with various (many) compile errors.

configure.ac: MinGW build fix (setting WINVER=0x0501).

Without this the build fails with:

  windows.c:40:43: error: 'WC_NO_BEST_FIT_CHARS' undeclared (first use in
  this function)

serialport.c: Fix an 'unused variable' compiler warning.

Avoid use of struct serial_struct on systems which don't have it (Android)

This fixes bug #376.

Don't use readlinkat if it does not exist

This fixes bug #377.

Introduce SP_API/SP_PRIV to mark visibility of functions.

Fix build for platforms without enumeration & port metadata support.

Move list_ports() implementations to OS-specific files.

Fix missing variable on macosx.

Ignore build leftovers.

Check bus:address and vid:pid validity.

Tidy up and split most OS-specific code to separate files.