fixed some selection bugs

Author: retailcoder

RetailCoder.VBE/Inspections/QualifiedModuleName.cs

@@ -18,5 +18,10 @@ public QualifiedModuleName(string project, string module)
 
         private readonly string _module;
         public string ModuleName { get { return _module; } }
+
+        public override string ToString()
+        {
+            return _project + "." + _module;
+        }
     }
 }

RetailCoder.VBE/UI/ToDoItems/ToDoExplorerDockablePresenter.cs

@@ -11,7 +11,9 @@
 
 namespace Rubberduck.UI.ToDoItems
 {
-    /// <summary>   (Not COM visible) Presenter for the Todo Explorer.  </summary>
+    /// <summary>
+    /// Presenter for the todo-items explorer.
+    /// </summary>
     [ComVisible(false)]
     public class ToDoExplorerDockablePresenter : DockablePresenterBase
     {

RetailCoder.VBE/VBA/Nodes/CommentNode.cs

@@ -17,5 +17,13 @@ public CommentNode(string comment, QualifiedSelection qualifiedSelection)
 
         private readonly QualifiedSelection _qualifiedSelection;
         public QualifiedSelection QualifiedSelection { get { return _qualifiedSelection; } }
+
+        /// <summary>
+        /// Returns the comment text.
+        /// </summary>
+        public override string ToString()
+        {
+            return _comment;
+        }
     }
 }

RetailCoder.VBE/VBA/VBParser.cs

@@ -80,7 +80,6 @@ public IEnumerable<CommentNode> ParseComments(VBComponent component)
             var continuing = false;
 
             var startLine = 0;
-            int startColumn;
 
             for (var i = 0; i < code.Length; i++)
             {
@@ -90,23 +89,25 @@ public IEnumerable<CommentNode> ParseComments(VBComponent component)
                 if (continuing || line.HasComment(out index))
                 {
                     startLine = continuing ? startLine : i;
-                    startColumn = index;
 
+                    var startColumn = index;
                     var commentLength = line.Length - index;
 
                     continuing = line.EndsWith("_");
                     if (!continuing)
                     {
-                        commentBuilder.Append(line.Substring(startColumn, commentLength).Trim());
-                        var selection = new Selection(startLine + 1, startColumn + 1, i, line.Length);
-                        yield return new CommentNode(commentBuilder.ToString(), new QualifiedSelection(qualifiedName, selection));
+                        commentBuilder.Append(line.Substring(startColumn, commentLength).TrimStart());
+                        var selection = new Selection(startLine + 1, startColumn + 1, i + 1, line.Length);
+
+                        var result = new CommentNode(commentBuilder.ToString(), new QualifiedSelection(qualifiedName, selection));
                         commentBuilder.Clear();
+                        
+                        yield return result;
                     }
                     else
                     {
-                        // ignore line continuations in comment text:
-
-                        commentBuilder.Append(line.Remove(line.Length - 1).TrimStart()); 
+                        // ignore line continuations in comment text; take commentLength - 1:
+                        commentBuilder.Append(line.Substring(startColumn, commentLength - 1).TrimStart()); 
                     }
                 }
             }