Brandon Williams <bmwill@xxxxxxxxxx> writes: > + /* If using a new version put that stuff here after a second null byte */ > + strbuf_addch(&request, '\0'); > + strbuf_addf(&request, "version=%d%c", 2, '\0'); > + /* subsequent supported versions can also be added */ > + strbuf_addf(&request, "version=%d%c", 3, '\0'); Isn't this last one meant only as a comment?