Resolve TODOs

src/Linkifier.ts

@@ -77,7 +77,6 @@ export class Linkifier {
 
     // Clear out any existing links
     this._mouseZoneManager.clearAll();
-    // TODO: Cancel any validation callbacks
 
     if (this._rowsTimeoutId) {
       clearTimeout(this._rowsTimeoutId);
@@ -91,6 +90,7 @@ export class Linkifier {
    * @param end The row to end at.
    */
   private _linkifyRows(start: number, end: number): void {
+    this._rowsTimeoutId = null;
     for (let i = start; i <= end; i++) {
       this._linkifyRow(i);
     }
@@ -220,8 +220,11 @@ export class Linkifier {
     // Ensure the link is valid before registering
     if (matcher.validationCallback) {
       matcher.validationCallback(text, isValid => {
+        // Discard link if the line has already changed
+        if (this._rowsTimeoutId) {
+          return;
+        }
         if (isValid) {
-          // TODO: Discard link if the line has already changed?
           this._addLink(offset + index, rowIndex, uri, matcher);
         }
       });

src/renderer/BaseRenderLayer.ts

@@ -14,8 +14,6 @@ export abstract class BaseRenderLayer implements IRenderLayer {
   private scaledLineHeight: number;
   private scaledLineDrawY: number;
 
-  // TODO: This should be shared between terminals, but not for static as some
-  // terminals may have different styles
   private _charAtlas: HTMLCanvasElement | ImageBitmap;
 
   constructor(
@@ -32,7 +30,6 @@ export abstract class BaseRenderLayer implements IRenderLayer {
     container.appendChild(this._canvas);
   }
 
-  // TODO: Should this do anything?
   public onOptionsChanged(terminal: ITerminal): void {}
   public onBlur(terminal: ITerminal): void {}
   public onFocus(terminal: ITerminal): void {}

src/renderer/Renderer.ts

@@ -33,13 +33,13 @@ export class Renderer implements IRenderer {
 
   public setTheme(theme: ITheme): IColorSet {
     this._colorManager.setTheme(theme);
+
     // Clear layers and force a full render
     this._renderLayers.forEach(l => {
       l.onThemeChanged(this._terminal, this._colorManager.colors);
       l.reset(this._terminal);
     });
 
-    // TODO: This is currently done for every single terminal, but it's static so it's wasting time
     this._terminal.refresh(0, this._terminal.rows - 1);
 
     return this._colorManager.colors;