Re: 'warning: unused parameter'
Markus Wanner <[email protected]>
| Newsgroups | gmane.comp.version-control.monotone.devel |
|---|---|
| Message-ID | <[email protected]> |
Stephen, On 05/01/2014 07:36 PM, Stephen Leake wrote: > Well, I started a separate file just in case. It turns out it is very > different; much simpler, since most of the dependent packages are > already packaged for MinGW64, and we can use 'pacman' to install them. pacman? Didn't read about that. Sounds good. The MinGW-w64 variant I fetched had *less* than the 32-bit one. Which one do you use? > The 64 bit changes are mostly due to Windows using a different > underlying type for socket_type than Unix. There are many changes; That worries me a tad. > anything that checks for an invalid socket, some function return or > parameter types. The code is definitely cleaner after the fixes, so I > don't think it will break anything on 32 bit. Phew, okay. Please try to keep the changes minimal. > I've already done most of the work, but not commited or run the tests > yet; I'll commit it on a branch. Can you get this done by Friday night? I planned to release this weekend. > I work much better when there are no spurious warnings; it's easier to > see the real problems in the compiler output. > > So I'll add -Wno-unused-parameter to suppress the warnings, and take it > out on the branch. You can always do `./configure CXXFLAGS="-Wno-unused-parameter"` - then there's nothing to take out. Regards Markus