• 
      

    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:
    Completed
    Change Summary:
    Pushed to release-2.0.x (8fc96dd)