Page MenuHomePhabricator

Use `ancestors(x)` instead of `0::x` in Mercurial history queries
ClosedPublic

Authored by epriestley on Dec 27 2013, 4:42 PM.
Tags
None
Referenced Files
F19088047: D7844.id.diff
Dec 3 2025, 2:58 AM
F19085568: D7844.diff
Dec 2 2025, 7:02 PM
F18851686: D7844.id.diff
Oct 31 2025, 3:07 AM
F18850598: D7844.diff
Oct 30 2025, 6:08 PM
F18105621: D7844.id17753.diff
Aug 10 2025, 6:56 PM
F18095194: D7844.diff
Aug 7 2025, 10:33 PM
F17816541: D7844.id17753.diff
Jul 25 2025, 10:50 PM
F17807160: D7844.id17742.diff
Jul 25 2025, 2:37 PM
Subscribers

Details

Summary

If 0 isn't an ancestor of the current branch, the 0::x construction fails. This is uncommon, but not wildly unreasonable. The ancestors() construction is simpler anyway.

Test Plan

Viewed some hg repos locally (change history, file history) without anything suspicious cropping up.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped