This is an archive of the discontinued Mercurial Phabricator instance.

filemerge: do what the context __bytes__ does, but locally
ClosedPublic

Authored by durin42 on Feb 26 2018, 12:16 AM.

Details

Summary

str() here is clearly the wrong thing, and I think the code is clearer
when it doesn't just depend on the magic {str,bytes} behavior.

I decided to grep around for \sstr\( and see what low-hanging fruit
that showed me. This was part of that hunt. That grep pattern still
has some things worth exploring.

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

durin42 created this revision.Feb 26 2018, 12:16 AM
indygreg accepted this revision.Feb 26 2018, 12:43 AM
indygreg added a subscriber: indygreg.

I agree that making this behavior explicit instead of relying on coercion is better. At least in this case.

This revision is now accepted and ready to land.Feb 26 2018, 12:43 AM
This revision was automatically updated to reflect the committed changes.