Run post-commit SVN diffs through convert_to_unicode.

Review Request #6763 — Created Jan. 12, 2015 and submitted

Information

Review Board
release-2.0.x
3af12e5...

Reviewers

We were blindly interpreting the results of the lower-level SVN libraries' diff
commands as utf-8. This worked until now, but it failed for a certain
iso-8859-1 character. This change switches us to use our fancy
convert_to_unicode method, which will try utf-8, anything that's configured for
the repository, and iso-8859-1.

Ran unit tests.

Description From Last Updated

Wouldn't it be better to let the implementations be dumb and just do the conversion here?

chipx86chipx86
There are no open issues
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        reviewboard/scmtools/svn/subvertpy.py
        reviewboard/scmtools/svn/pysvn.py
        reviewboard/scmtools/svn/__init__.py
        reviewboard/scmtools/svn/base.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/scmtools/svn/subvertpy.py
        reviewboard/scmtools/svn/pysvn.py
        reviewboard/scmtools/svn/__init__.py
        reviewboard/scmtools/svn/base.py
    
    
  2. 
      
david
chipx86
  1. 
      
  2. reviewboard/scmtools/svn/__init__.py (Diff revision 1)
     
     
     
     
    Show all issues

    Wouldn't it be better to let the implementations be dumb and just do the conversion here?

  3. 
      
david
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        reviewboard/scmtools/svn/subvertpy.py
        reviewboard/scmtools/svn/__init__.py
        reviewboard/scmtools/svn/pysvn.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/scmtools/svn/subvertpy.py
        reviewboard/scmtools/svn/__init__.py
        reviewboard/scmtools/svn/pysvn.py
    
    
  2. 
      
chipx86
  1. Ship It!
  2. 
      
david
Review request changed
Status:
Completed
Change Summary:
Pushed to release-2.0.x (ad8ccf1)
Loading...