Cache the full diff files list rather than only chunks

Review Request #139 — Created Aug. 10, 2007 and submitted

Information

Review Board SVN (deprecated)
trunk

Reviewers

We were caching the chunks that made up the get_diff_files list, when the only time the chunks were useful were in get_diff_files. So let's cache the results of that instead. Same with the file coming from the SCM backend, in order to make more room for the get_diff_files cache. In theory, if this falls out of the cache, we'll probably have lost the file contents anyway. Besides, we weren't caching HEAD files anyway.
Caching seems to work in the diff viewer and review request page with comments.
david
  1. Looks OK.  I see this is already in SVN.
  2. /trunk/reviewboard/diffviewer/views.py (Diff revision 2)
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
     
    Hrm.  This is kind of confusing, being so far down from the actual function definition.  I wonder if it might make sense to move those internal functions out now...
  3. 
      
DA
  1. 
      
  2.