D3371: scmutil: make shortesthexnodeidprefix() take a full binary nodeid

2018-04-15 Thread martinvonz (Martin von Zweigbergk)
This revision was automatically updated to reflect the committed changes. Closed by commit rHG1f473df0dffe: scmutil: make shortesthexnodeidprefix() take a full binary nodeid (authored by martinvonz, committed by ). CHANGED PRIOR TO COMMIT https://phab.mercurial-scm.org/D3371?vs=8286=8303#toc

D3371: scmutil: make shortesthexnodeidprefix() take a full binary nodeid

2018-04-15 Thread yuja (Yuya Nishihara)
yuja added a comment. Queued thanks. > The shortest() template function depended on the behavior of > revlog._partialmatch() for these types of inputs: I guess some of them would be bug. INLINE COMMENTS > templatefuncs.py:605 > +except (error.LookupError,

D3371: scmutil: make shortesthexnodeidprefix() take a full binary nodeid

2018-04-14 Thread martinvonz (Martin von Zweigbergk)
martinvonz updated this revision to Diff 8286. REPOSITORY rHG Mercurial CHANGES SINCE LAST UPDATE https://phab.mercurial-scm.org/D3371?vs=8256=8286 REVISION DETAIL https://phab.mercurial-scm.org/D3371 AFFECTED FILES hgext/show.py mercurial/scmutil.py mercurial/templatefuncs.py

D3371: scmutil: make shortesthexnodeidprefix() take a full binary nodeid

2018-04-14 Thread martinvonz (Martin von Zweigbergk)
martinvonz created this revision. Herald added a subscriber: mercurial-devel. Herald added a reviewer: hg-reviewers. REVISION SUMMARY The shortest() template function depended on the behavior of revlog._partialmatch() for these types of inputs: - non-hex strings - ambiguous strings -