[PATCH 07 of 10] chgserver: implement validate command

Yuya Nishihara yuya at tcha.org
Fri Mar 4 10:39:32 EST 2016


On Fri, 4 Mar 2016 14:17:06 +0000, Jun Wu wrote:
> On 03/04/2016 01:46 PM, Yuya Nishihara wrote:
> > I'm okay with "args". My point is that "args" isn't terminated by '\0', so
> > "insts" shouldn't be terminated by '\0' as well.
> 
> I see. But that will require some extra efforts in both client and server to
> make it work. I would be happy to keep it as is.

Hmm, I did that extra effort for "getpager" command. Since the client have to
make sure the received data is NULL terminated, there wouldn't be much
difference in C code.


More information about the Mercurial-devel mailing list