stephane / libmodbus

A Modbus library for Linux, Mac OS, FreeBSD and Windows
http://libmodbus.org
GNU Lesser General Public License v2.1
3.33k stars 1.71k forks source link

[PATCH] fix compilation #52

Closed oldfaber closed 12 years ago

oldfaber commented 12 years ago

A semicolon is missing. Please apply.

Best regards.

oldfaber

--- libmodbus-3.0.2\src\modbus-rtu.c 2012-01-25 05:29:36.000000000 +0100 +++ libmodbus-3.0.2-new\src\modbus-rtu.c 2012-04-17 15:22:20.640656500 +0200 @@ -351,7 +351,7 @@ if (!GetCommState(ctx_rtu->w_ser.fd, &ctx_rtu->old_dcb)) { fprintf(stderr, "ERROR Error getting configuration (LastError %d)\n", (int)GetLastError());

stephane commented 12 years ago

Already fixed in master and stable branches, why are you patching the tarball?!

oldfaber commented 12 years ago

Because I don't speak git? I also have more minor fixes for Microsoft compiler and bigger changes for creating a Windows DLL from the library. Could you accept patches ? I will learn git, but have no time now.

Regards Fabio

stephane commented 12 years ago

Yes I accept patches of course but you can download the latest code as tarball/zip from github to avoid to work on old code.