diff mercurial/mpatch.c @ 26686:3e7f675628ad

wireproto: properly parse false boolean args (BC) The client represents boolean arguments as '0' and '1'. bool('0') == bool('1') == True, so a simple bool(val) isn't sufficient for converting the argument back to a bool type. Currently, "obsmarkers" is the only boolean argument to getbundle. I /think/ the only place where we currently set the "obsmarkers" argument is during bundle2 pulls. As a result of this bug, the server /might/ be sending obsolete markers bundle2 part(s) to clients that don't request them. That is why I marked this BC. Surprisingly there was no test fall out from this change. I suspect a lapse in test coverage.
author Gregory Szorc <gregory.szorc@gmail.com>
date Wed, 14 Oct 2015 10:58:35 -0700
parents 09e41ac6289d
children b6ed2505d6cf
line wrap: on
line diff