Always parse diffs as binary.

Review Request #6926 — Created Feb. 9, 2015 and submitted

Information

Review Board
release-2.0.x
f6c057f...

Reviewers

In our diff parsing code, we would convert to unicode first, and then parse in
unicode mode. This wasn't ideal, because things like splitlines() behave
differently for unicode, and once we add support for diffX, we'll need to parse
in binary anyway (since each file will have different encodings).

  • Ran unit tests.
  • Uploaded a diff with some unicode characters.
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        reviewboard/diffviewer/managers.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/diffviewer/managers.py
    
    
  2. 
      
chipx86
  1. Ship It!
  2. 
      
david
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-2.0.x (8fc96dd)
Loading...