[PATCH 5 of 9 V6] bookmarks: add srchex param to updatefromremote

Pierre-Yves David pierre-yves.david at ens-lyon.org
Fri Nov 11 06:59:12 EST 2016



On 11/11/2016 11:29 AM, Stanislau Hlebik wrote:
> Not sure I fully understand your idea.
> Do you suggest to rewrite bookmarks.compare and bookmarks.updatefromremote to accept only binary nodes?

Yes, it would like it would make more sense.

If I understand your series correctly, we otherwise get in a situation 
where a caller with the binary data has to convert them to hex and the 
function internal have to convert thing again because internal storage 
is node. If everybody were talking node the situation would be simpler.

Does this make sense ?

-- 
Pierre-Yves David


More information about the Mercurial-devel mailing list