comments: place the left over comments (outdated/misplaced) to the left or right pane.

So we don't loose the original context where they were placed.

rhodecode/lib/codeblocks.py

@@ -402,6 +402,7 b' class DiffSet(object):'
         return diffset
 
     _lexer_cache = {}
+
     def _get_lexer_for_filename(self, filename, filenode=None):
         # cached because we might need to call it twice for source/target
         if filename not in self._lexer_cache:
@@ -501,6 +502,9 b' class DiffSet(object):'
         if target_file_path in self.comments_store:
             for lineno, comments in self.comments_store[target_file_path].items():
                 left_comments[lineno] = comments
+        # left comments are one that we couldn't place in diff lines.
+        # could be outdated, or the diff changed and this line is no
+        # longer available
         filediff.left_comments = left_comments
 
         return filediff
@@ -608,11 +612,11 b' class DiffSet(object):'
 
         return lines
 
-    def get_comments_for(self, version, file, line_number):
-        if hasattr(file, 'unicode_path'):
-            file = file.unicode_path
+    def get_comments_for(self, version, filename, line_number):
+        if hasattr(filename, 'unicode_path'):
+            filename = filename.unicode_path
 
-        if not isinstance(file, basestring):
+        if not isinstance(filename, basestring):
             return None
 
         line_key = {
@@ -620,8 +624,8 b' class DiffSet(object):'
             'new': 'n',
         }[version] + str(line_number)
 
-        if file in self.comments_store:
-            file_comments = self.comments_store[file]
+        if filename in self.comments_store:
+            file_comments = self.comments_store[filename]
             if line_key in file_comments:
                 return file_comments.pop(line_key)
 

rhodecode/templates/codeblocks/diffs.mako

@@ -235,12 +235,18 b' collapse_all = len(diffset.files) > coll'
             <tr class="cb-line">
                 <td class="cb-data cb-context"></td>
                 <td class="cb-lineno cb-context"></td>
-                <td class="cb-content cb-context"></td>
+                <td class="cb-content cb-context">
+                    % if lineno.startswith('o'):
+                        ${inline_comments_container(comments)}
+                    % endif
+                </td>
 
                 <td class="cb-data cb-context"></td>
                 <td class="cb-lineno cb-context"></td>
                 <td class="cb-content cb-context">
-                    ${inline_comments_container(comments)}
+                    % if lineno.startswith('n'):
+                        ${inline_comments_container(comments)}
+                    % endif
                 </td>
             </tr>
         %endif