Compilation error with --enable-pedantic

[stipa]

With --enable-pedantic option enabled, compilation fails in compat-lz4.c:

compat-lz4.c:154:1: error: expected identifier or '(' before '/' token
 //**************************************
 ^

 compat-lz4.c:154:2: error: unterminated comment
 //**************************************
  ^

[Gert Döring]

thanks. We've imported a somewhat oldish version of lz4.c from upstream, which wants a refresh anyway - so, I'll do, and check for these sort of "non-pretty" comments at that opportunity.

Relevant for 2.4, as 2.3 doesn't have lz4 at all.

(Workaround: install lz4 externally, where openvpn configure can find it, do not use the compat stuff :) )

[Gert Döring]

a more recent version of the LZ4 sources has now been imported (r131)

commit 46e4b6639a950c56a08261b6610e7fe17404213d
Author: Gert Doering <gert@…>
Date: Thu Jun 9 13:51:29 2016 +0200

Upgrade bundled compat-lz4 to upstream release r131.

... but that still does not work with --enable-pedantic, as it uses "long long" which is not in C90. To make it work, change "-ansi" to "-std=c99".

A new patch for that will be sent shortly.

[Gert Döring]

Haha, that was quick :-)

commit d16072cf17ce8debcf796565841a54f1253a9923
Author: Gert Doering <gert@…>
Date: Thu Jun 9 15:00:32 2016 +0200

Change --enable-pedantic to use -std=c99 and not -ansi (C90).

with that change, we still have a number of warnings related to -Wstrict-aliasing (which we can't do much about, as it's all sockaddr stuff) and a few signed/unsigned mismatches. But at least everything compiles :-)