Merge pull request emberjs#16264 from emberjs/more-classes

Convert Descriptor and Alias to native classes
Dhaulagiri · Feb 21, 2018 · 67bf23f · 67bf23f
2 parents 7703d4f + 45b9f15
commit 67bf23f
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 7 deletions.
diff --git a/packages/ember-metal/lib/mixin.js b/packages/ember-metal/lib/mixin.js
@@ -656,13 +656,13 @@ export function required() {
   return REQUIRED;
 }
 
-function Alias(methodName) {
-  this.isDescriptor = true;
-  this.methodName = methodName;
+class Alias extends Descriptor {
+  constructor(methodName) {
+    super();
+    this.methodName = methodName;
+  }
 }
 
-Alias.prototype = new Descriptor();
-
 /**
   Makes a method available via an additional name.
 

diff --git a/packages/ember-metal/lib/properties.js b/packages/ember-metal/lib/properties.js
@@ -19,8 +19,10 @@ import { peekCacheFor } from './computed';
   @class Descriptor
   @private
 */
-export function Descriptor() {
-  this.isDescriptor = true;
+export class Descriptor {
+  constructor() {
+    this.isDescriptor = true;
+  }
 }
 
 // ..........................................................