search

fpagliughi / sockpp

Modern C++ socket library.
BSD 3-Clause "New" or "Revised" License
769 stars 126 forks source link

Fixup: Removed trailing ';' #62

Closed ramonnr closed 2 years ago

ramonnr commented 2 years ago

Which triggers warning when compiling with -Wall

ramonnr commented 2 years ago

Getting the warning with g++ 10.2.1-6, which as of this writing is the compiler shipped in build-essential on debian-stable

fpagliughi commented 2 years ago

Thanks!

Strangely, I tried it with g++ 10.3 and 11.1 on Ubuntu 20.04, and couldn't reproduce the warning, but certainly the semicolon is unnecessary.

ramonnr commented 2 years ago

I mixed up the flags, it's -pedantic which triggers a warning on extra ';'.

On another subject, any idea on when the next merge to master / release will happen? Looking forward to dropping my fork the minute it happens :D

fpagliughi commented 2 years ago

Should be today...

ramonnr commented 2 years ago

Woho!