chore(compiler): dead code clean-up.

rkirov · rkirov · commit bfe3efab3b2d · 2015-10-27T12:00:44.000Z
Dead code that uglifyJS found. Closes angular#4907
diff --git a/modules/angular2/src/core/change_detection/dynamic_change_detector.ts b/modules/angular2/src/core/change_detection/dynamic_change_detector.ts
@@ -391,10 +391,7 @@ export class DynamicChangeDetector extends AbstractChangeDetector<any> {
   _readContext(proto: ProtoRecord, values: any[]) {
     if (proto.contextIndex == -1) {
       return this._getDirectiveFor(proto.directiveIndex);
-    } else {
-      return values[proto.contextIndex];
     }
-
     return values[proto.contextIndex];
   }
 
diff --git a/modules/angular2/src/core/compiler/html_parser.ts b/modules/angular2/src/core/compiler/html_parser.ts
@@ -19,8 +19,6 @@ import {
 import {escapeDoubleQuoteString} from './util';
 import {Injectable} from 'angular2/src/core/di';
 
-const NG_NON_BINDABLE = 'ng-non-bindable';
-
 @Injectable()
 export class HtmlParser {
   parse(template: string, sourceInfo: string): HtmlAst[] {
diff --git a/modules/angular2/src/core/compiler/runtime_metadata.ts b/modules/angular2/src/core/compiler/runtime_metadata.ts
@@ -20,10 +20,6 @@ import {reflector} from 'angular2/src/core/reflection/reflection';
 import {Injectable} from 'angular2/src/core/di';
 import {MODULE_SUFFIX} from './util';
 
-// group 1: "property" from "[property]"
-// group 2: "event" from "(event)"
-var HOST_REG_EXP = /^(?:(?:\[([^\]]+)\])|(?:\(([^\)]+)\)))$/g;
-
 @Injectable()
 export class RuntimeMetadataResolver {
   private _cache = new Map<Type, cpl.CompileDirectiveMetadata>();
diff --git a/modules/angular2/src/core/compiler/template_parser.ts b/modules/angular2/src/core/compiler/template_parser.ts
@@ -628,10 +628,6 @@ class BoundElementOrDirectiveProperty {
               public sourceInfo: string) {}
 }
 
-class ParseError {
-  constructor(public message: string, public sourceInfo: string) {}
-}
-
 export function splitClasses(classAttrValue: string): string[] {
   return StringWrapper.split(classAttrValue.trim(), /\s+/g);
 }
diff --git a/modules/angular2/src/core/render/dom/dom_renderer.ts b/modules/angular2/src/core/render/dom/dom_renderer.ts
@@ -322,15 +322,6 @@ function moveNodesAfterSibling(sibling, nodes) {
   }
 }
 
-function moveChildNodes(source: Node, target: Node) {
-  var currChild = DOM.firstChild(source);
-  while (isPresent(currChild)) {
-    var nextChild = DOM.nextSibling(currChild);
-    DOM.appendChild(target, currChild);
-    currChild = nextChild;
-  }
-}
-
 function decoratePreventDefault(eventHandler: Function): Function {
   return (event) => {
     var allowDefaultBehavior = eventHandler(event);
diff --git a/modules/angular2/src/core/render/view_factory.ts b/modules/angular2/src/core/render/view_factory.ts
@@ -252,9 +252,3 @@ class Component<N> {
                [];
   }
 }
-
-function addAll(source: any[], target: any[]) {
-  for (var i = 0; i < source.length; i++) {
-    target.push(source[i]);
-  }
-}

Original file line number	Diff line number	Diff line change
`@@ -391,10 +391,7 @@ export class DynamicChangeDetector extends AbstractChangeDetector<any> {`
`391`	`391`	`_readContext(proto: ProtoRecord, values: any[]) {`
`392`	`392`	`if (proto.contextIndex == -1) {`
`393`	`393`	`return this._getDirectiveFor(proto.directiveIndex);`
`394`		`- } else {`
`395`		`- return values[proto.contextIndex];`
`396`	`394`	`}`
`397`		`-`
`398`	`395`	`return values[proto.contextIndex];`
`399`	`396`	`}`
`400`	`397`
Original file line number	Diff line number	Diff line change
`@@ -628,10 +628,6 @@ class BoundElementOrDirectiveProperty {`
`628`	`628`	`public sourceInfo: string) {}`
`629`	`629`	`}`
`630`	`630`
`631`		`-class ParseError {`
`632`		`- constructor(public message: string, public sourceInfo: string) {}`
`633`		`-}`
`634`		`-`
`635`	`631`	`export function splitClasses(classAttrValue: string): string[] {`
`636`	`632`	`return StringWrapper.split(classAttrValue.trim(), /\s+/g);`
`637`	`633`	`}`
Original file line number	Diff line number	Diff line change
`@@ -252,9 +252,3 @@ class Component<N> {`
`252`	`252`	`[];`
`253`	`253`	`}`
`254`	`254`	`}`
`255`		`-`
`256`		`-function addAll(source: any[], target: any[]) {`
`257`		`- for (var i = 0; i < source.length; i++) {`
`258`		`- target.push(source[i]);`
`259`		`- }`
`260`		`-}`