cleanup

Author: AndreasArvidsson

src/libs/common/types/TextDocument.ts

@@ -46,8 +46,6 @@ export interface TextDocument {
    * that the returned object is *not* live and changes to the
    * document are not reflected.
    *
-   * The position will be {@link TextDocument.validatePosition adjusted}.
-   *
    * @see {@link TextDocument.lineAt}
    *
    * @param position A position.
@@ -58,8 +56,6 @@ export interface TextDocument {
   /**
    * Converts the position to a zero-based offset.
    *
-   * The position will be {@link TextDocument.validatePosition adjusted}.
-   *
    * @param position A position.
    * @return A valid zero-based offset.
    */
@@ -75,7 +71,7 @@ export interface TextDocument {
 
   /**
    * Get the text of this document. A substring can be retrieved by providing
-   * a range. The range will be {@link TextDocument.validateRange adjusted}.
+   * a range.
    *
    * @param range Include only the text included by the range.
    * @return The text inside the provided range or the entire text.

src/util/targetUtils.ts

@@ -54,8 +54,8 @@ function groupForEachEditor<T>(
   getEditor: (target: T) => TextEditor,
 ): [TextEditor, T[]][] {
   // Actually group by document and not editor. If the same document is open in multiple editors we want to perform all actions in one editor or an concurrency error will occur.
-  const getDocument = (target: T) => getEditor(target).document.uri;
-  const editorMap = groupBy(targets, getDocument);
+  const getDocumentUri = (target: T) => getEditor(target).document.uri;
+  const editorMap = groupBy(targets, getDocumentUri);
   return Array.from(editorMap.values(), (editorTargets) => {
     // Just pick any editor with the given document open; doesn't matter which
     const editor = getEditor(editorTargets[0]);