From 005b8583f5c744d7e8797d1072245e608196df2f Mon Sep 17 00:00:00 2001
From: Arthur Ozga <aozgaa@umich.edu>
Date: Fri, 17 Jul 2015 10:33:07 -0700
Subject: [PATCH] remove for-loops

---
 src/compiler/emitter.ts  | 10 ++++++++++
 src/services/services.ts | 28 ++++++++++++++++------------
 2 files changed, 26 insertions(+), 12 deletions(-)

diff --git a/src/compiler/emitter.ts b/src/compiler/emitter.ts
index 9c8438565e7..6073c0c9421 100644
--- a/src/compiler/emitter.ts
+++ b/src/compiler/emitter.ts
@@ -6764,6 +6764,16 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, Promi
                 return leadingComments;
             }
 
+            /**
+             * Removes all but the pinned or triple slash comments.
+             * @param ranges The array to be filtered
+             * @param onlyPinnedOrTripleSlashComments whether the filtering should be performed.
+             * 
+             * This probably shouldn't be a parameter at all. It appears that in every call, the argument is
+             * precisely 'compilerOptions.removeComments'.
+             * 
+             * How to fix this?
+             */
             function filterComments(ranges: CommentRange[], onlyPinnedOrTripleSlashComments: boolean): CommentRange[] {
                 // If we're removing comments, then we want to strip out all but the pinned or
                 // triple slash comments.
diff --git a/src/services/services.ts b/src/services/services.ts
index c2432816f24..71dc03f35ce 100644
--- a/src/services/services.ts
+++ b/src/services/services.ts
@@ -260,26 +260,30 @@ namespace ts {
         }
 
         public getFirstToken(sourceFile?: SourceFile): Node {
-            let children = this.getChildren();
-            for (let child of children) {
-                if (child.kind < SyntaxKind.FirstNode) {
-                    return child;
-                }
+            let children = this.getChildren(); // why isn't sourceFile passed as an argument??
+            if (!(children && children.length > 0)) { return undefined; }
 
-                return child.getFirstToken(sourceFile);
+            let child = children[0];
+
+            if (child.kind < SyntaxKind.FirstNode) {
+                return child;
             }
+
+            return child.getFirstToken(sourceFile);
         }
 
         public getLastToken(sourceFile?: SourceFile): Node {
             let children = this.getChildren(sourceFile);
-            for (let i = children.length - 1; i >= 0; i--) {
-                let child = children[i];
-                if (child.kind < SyntaxKind.FirstNode) {
-                    return child;
-                }
+            if (!children) { return undefined; }
 
-                return child.getLastToken(sourceFile);
+            let child = lastOrUndefined(children);
+            if (!child) { return undefined; }
+
+            if (child.kind < SyntaxKind.FirstNode) {
+                return child;
             }
+
+            return child.getLastToken(sourceFile);
         }
     }