On 10/11/2016 06:25 PM, Stanislau Hlebik wrote:
# HG changeset patch # User Stanislau Hlebik <st...@fb.com> # Date 1476195835 25200 # Tue Oct 11 07:23:55 2016 -0700 # Node ID 91ae660eb2faf74cb96ee7ee5bdcc1f2507f9cf7 # Parent 55e997127023d7208488c593adb933a1bfb23312 bookmarks: introduce BookmarksEncodeError Binary encoding for bookmarks will be added in next diffs. This error will be thrown if there are problems with bookmarks encoding.
Other code I can find about encoding/decoding use ValueError in case of trouble. We can probably stick to that for now.
We don't usually introduce very specialized exception unless they have a very specialized purpose. From looking at the rest of the series I do not see anything special about there usage (but I might have missed something).
Cheers -- Pierre-Yves David _______________________________________________ Mercurial-devel mailing list Mercurial-devel@mercurial-scm.org https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel