# HG changeset patch # User Augie Fackler # Date 1533882614 14400 # Node ID ed8160e4fea0480301b241fd967d4fa0ba62cad7 # Parent 2e37bd6148797fc50f07d7d23ab8d3c0fa005e87 util: fix signature of setsockopt in socket observer The format string expected the result, callers all passed the result, but we didn't actually accept the result to this function. I swear I've looked at this code a dozen times trying to find the problem, and it was this simple all along. Differential Revision: https://phab.mercurial-scm.org/D4253 diff -r 2e37bd614879 -r ed8160e4fea0 mercurial/util.py --- a/mercurial/util.py Fri Aug 10 02:03:26 2018 -0400 +++ b/mercurial/util.py Fri Aug 10 02:30:14 2018 -0400 @@ -948,12 +948,12 @@ self.fh.write('%s> gettimeout() -> %f\n' % (self.name, res)) - def setsockopt(self, level, optname, value): + def setsockopt(self, res, level, optname, value): if not self.states: return self.fh.write('%s> setsockopt(%r, %r, %r) -> %r\n' % ( - self.name, level, optname, value)) + self.name, level, optname, value, res)) def makeloggingsocket(logh, fh, name, reads=True, writes=True, states=True, logdata=False, logdataapis=True):