This is an archive of the discontinued Mercurial Phabricator instance.

alias: test duplicated definition earlier
ClosedPublic

Authored by quark on Sep 23 2017, 4:57 PM.

Details

Summary

This patch moves the old definition checking logic introduced by
f4b7be3f8430 earlier. So that the test itself does not depend on aliasdef.

The check is to avoid wrapping a same alias multiple times. It can be done
by checking the config name and value (definition in code), without
constructing a cmdalias instance.

This makes the next patch easier to review.

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

quark created this revision.Sep 23 2017, 4:57 PM
quark updated this revision to Diff 2051.Sep 23 2017, 5:41 PM
This revision was automatically updated to reflect the committed changes.