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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.