D3665: graph: improve graph output by using Unicode characters

yuja (Yuya Nishihara) phabricator at mercurial-scm.org
Tue May 29 09:22:12 EDT 2018


yuja added a comment.


  > except for `encoding._wide`. I am not sure what your expectation is for that.
  >  I don't think it would make sense to silently disable the extension if
  >  `encoding._wide` is set; IMO that would cause user confusion.
  
  I think that's also true for `encoding != 'UTF-8'`. The simplest solution
  would be to show a warning when loading the extension.
  
    def extsetup(ui):
        if encoding.encoding != 'UTF-8' or encoding._wide:
            ui.warn(_('unsupported encoding or terminal setting\n'))
            return

REPOSITORY
  rHG Mercurial

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

To: johnstiles, #hg-reviewers
Cc: yuja, mercurial-devel


More information about the Mercurial-devel mailing list