martinvonz added a comment.

  In https://phab.mercurial-scm.org/D3536#56022, @indygreg wrote:
  
  > `unicode` is not a type on Python 3. I think this should be changed to 
`(str, pycompat.unicode)`. That will evaluate to `(str, unicode)` on Python 2 
and `(str, str)` on Python 3. If you go this route, please rephrase the comment 
to denote the Python 2 behavior only triggering on `unicode` because of `str == 
bytes`.
  
  
  Done, I believe.

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D3536

To: martinvonz, #hg-reviewers, indygreg
Cc: yuja, indygreg, mercurial-devel
_______________________________________________
Mercurial-devel mailing list
Mercurial-devel@mercurial-scm.org
https://www.mercurial-scm.org/mailman/listinfo/mercurial-devel

Reply via email to