code review

Author: FabriceSalvaire

DiffViewer/SyntaxHighlighter.py

@@ -9,7 +9,7 @@
 #
 #                                              Audit
 #
-# - 02/04/2012 Fabrice
+# - 04/06/2012 Fabrice
 #   Check design: purpose of HighlightedText ? Merge code?
 #
 ####################################################################################################
@@ -64,7 +64,7 @@ class HighlightedText(list):
     def __init__(self, raw_text_document, lexer):
 
         """ The parameter *raw_text_document* is a :class:`DiffViewer.RawTextDocument` instance and
-        the parameter *lexer* is Pygments lexer instance.
+        the parameter *lexer* is a Pygments lexer instance.
         """
 
         super(HighlightedText, self).__init__()
@@ -93,10 +93,10 @@ def highlight_text(raw_text_document, lexer):
     """ Highlight a text.
 
     The parameter *raw_text_document* is a :class:`DiffViewer.RawTextDocument` instance and the
-    parameter *lexer* is Pygments lexer instance.
+    parameter *lexer* is a Pygments lexer instance.
 
-    Return an :class:`TextDocumentModel` instance.  The document has one text block that contains
-    all the fragments.  Text fragments use light views.
+    Return an :class:`DiffViewer.TextDocumentModel` instance.  The document has one text block that
+    contains all the fragments.  Text fragments use light views.
     """
 
     highlighted_text = HighlightedText(raw_text_document, lexer)

DiffViewer/diff-viewer

@@ -74,20 +74,28 @@ class Application(QtGui.QApplication):
             self._raw_text_documents.append(raw_text_document)
             lexer = get_lexer_for_filename(file_name, stripnl=False)
             self._lexers.append(lexer)
-            
+
+        self._highlighted_documents = []
         if not args.show:
             file_diff = TwoWayFileDiffFactory().process(* self._raw_text_documents)
             self._document_models = TextDocumentDiffModelFactory().process(file_diff)
             self._highlighted_texts = []
             for raw_text_document, lexer in zip(self._raw_text_documents, self._lexers):
                 highlighted_text = HighlightedText(raw_text_document, lexer)
                 self._highlighted_texts.append(highlighted_text)
-            self._highlighted_documents = []
             for document_model, highlighted_text in zip(self._document_models, self._highlighted_texts):
                 highlighted_document = highlight_document(document_model, highlighted_text)
                 self._highlighted_documents.append(highlighted_document)
         else:
-            self._highlighted_documents = []
+            # Fixme: broken
+            # self._diff_view.set_document_models(self._highlighted_documents, complete_mode)
+            # File "/home/gv/fabrice/unison-osiris/git-python/DiffViewer/DiffWidget.py", line 333, in set_document_models
+            # cursor.begin_block(side, text_block.frame_type)
+            # File "/home/gv/fabrice/unison-osiris/git-python/DiffViewer/DiffWidget.py", line 99, in begin_block
+            # if ((side == LEFT and frame_type == chunk_type.insert) or
+            # File "/home/gv/fabrice/unison-osiris/git-python/DiffViewer/Tools/EnumFactory.py", line 107, in __eq__
+            # return self._value == int(other)
+            # TypeError: int() argument must be a string or a number, not 'NoneType'
             for raw_text_document, lexer in zip(self._raw_text_documents, self._lexers):
                 highlighted_document = highlight_text(raw_text_document, lexer)
                 self._highlighted_documents.append(highlighted_document)

review-log.txt

@@ -15,7 +15,7 @@ DiffViewer.Tools
  + TextDocumentModel.py
  ? RawTextDocumentDiff.py : see audit
  ? SyntaxHighlighter.py : see audit
- - TextDocumentDiffModel.py : review and complete !
+ ? TextDocumentDiffModel.py : check code
  x DiffWidget.py
  x __init__.py
  x diff-viewer

setenv.sh

@@ -2,6 +2,7 @@
 
 ####################################################################################################
 
+append_to_ld_library_path_if_not /usr/local/lib
 append_to_python_path_if_not ${PWD}
 
 ####################################################################################################

test/git-python

@@ -1,6 +1,6 @@
 #! /usr/bin/env python
 
-#####################################################################################################
+####################################################################################################
 
 from __future__ import print_function
 
@@ -9,7 +9,7 @@ import os
 import sys
 import time
 
-#####################################################################################################
+####################################################################################################
 
 if len(sys.argv) != 2:
     print("Give a repository path")
@@ -23,7 +23,7 @@ repository_path = os.path.normpath(repository_path)
 
 print("Repository:", repository_path)
 
-#####################################################################################################
+####################################################################################################
 
 repository = git.Repo(repository_path)
 
@@ -71,7 +71,7 @@ Commit:
             print('-'*25)
             print(entry.data_stream.read())
 
-#####################################################################################################
+####################################################################################################
 
 print('*'*80)
 print('Diff:')
@@ -84,8 +84,8 @@ for diff_type in 'A', 'D', 'R', 'M':
         print(diff_type)
         print(entry)
 
-#####################################################################################################
+####################################################################################################
 #
 # End
 #
-#####################################################################################################
+####################################################################################################