[PATCH 2 of 3 V3] schemas: add schemas listkeys implementation
Martin Geisler
martin at geisler.net
Thu Aug 15 03:04:12 CDT 2013
Durham Goode <durham at fb.com> writes:
> # HG changeset patch
> # User Durham Goode <durham at fb.com>
> # Date 1375827497 25200
> # Tue Aug 06 15:18:17 2013 -0700
> # Node ID d43e79e852d560c9990aa2ac2de878c73e9b88ad
> # Parent 84ac4b920802805474d203b5b5dc1cd4aa861256
> schemas: add schemas listkeys implementation
>
> This adds the wire protocol logic for requesting and transmitting
> schema data. It is transmitted via the existing listkeys protocol,
> but schemas cannot be pushed to the server.
Oh, it seems you do use the listkey system :) Great!
What confused me was that you talk about adding logic to the wire
protocol when you're really just using the existing protocol without
extending it.
In other words: this reviewer has a very short attention span and poor
read-ahead capabilities :) So think about spelling out the obvious and
highlight when you're using existing infrastructure (that automatically
makes your patch seem more elegant).
--
Martin Geisler
More information about the Mercurial-devel
mailing list