Revert change to abstract-method-with-body error location

Iain Monro · Iain Monro · commit 46f1a2cc5dcd · 2015-12-08T12:25:45.000Z
diff --git a/src/compiler/checker.ts b/src/compiler/checker.ts
@@ -11175,7 +11175,7 @@ namespace ts {
             // Abstract methods cannot have an implementation.
             // Extra checks are to avoid reporting multiple errors relating to the "abstractness" of the node.
             if (node.flags & NodeFlags.Abstract && node.body) {
-                error(node.body, Diagnostics.Method_0_cannot_have_an_implementation_because_it_is_marked_abstract, declarationNameToString(node.name));
+                error(node, Diagnostics.Method_0_cannot_have_an_implementation_because_it_is_marked_abstract, declarationNameToString(node.name));
             }
         }
 
diff --git a/tests/baselines/reference/classAbstractMethodInNonAbstractClass.errors.txt b/tests/baselines/reference/classAbstractMethodInNonAbstractClass.errors.txt
@@ -1,6 +1,6 @@
 tests/cases/conformance/classes/classDeclarations/classAbstractKeyword/classAbstractMethodInNonAbstractClass.ts(2,5): error TS1244: Abstract methods can only appear within an abstract class.
 tests/cases/conformance/classes/classDeclarations/classAbstractKeyword/classAbstractMethodInNonAbstractClass.ts(6,5): error TS1244: Abstract methods can only appear within an abstract class.
-tests/cases/conformance/classes/classDeclarations/classAbstractKeyword/classAbstractMethodInNonAbstractClass.ts(6,20): error TS1245: Method 'foo' cannot have an implementation because it is marked abstract.
+tests/cases/conformance/classes/classDeclarations/classAbstractKeyword/classAbstractMethodInNonAbstractClass.ts(6,14): error TS1245: Method 'foo' cannot have an implementation because it is marked abstract.
 
 
 ==== tests/cases/conformance/classes/classDeclarations/classAbstractKeyword/classAbstractMethodInNonAbstractClass.ts (3 errors) ====
@@ -14,6 +14,6 @@ tests/cases/conformance/classes/classDeclarations/classAbstractKeyword/classAbst
         abstract foo() {}
         ~~~~~~~~
 !!! error TS1244: Abstract methods can only appear within an abstract class.
-                       ~~
+                 ~~~
 !!! error TS1245: Method 'foo' cannot have an implementation because it is marked abstract.
     }
diff --git a/tests/baselines/reference/classAbstractMethodWithImplementation.errors.txt b/tests/baselines/reference/classAbstractMethodWithImplementation.errors.txt
@@ -1,9 +1,9 @@
-tests/cases/conformance/classes/classDeclarations/classAbstractKeyword/classAbstractMethodWithImplementation.ts(2,20): error TS1245: Method 'foo' cannot have an implementation because it is marked abstract.
+tests/cases/conformance/classes/classDeclarations/classAbstractKeyword/classAbstractMethodWithImplementation.ts(2,14): error TS1245: Method 'foo' cannot have an implementation because it is marked abstract.
 
 
 ==== tests/cases/conformance/classes/classDeclarations/classAbstractKeyword/classAbstractMethodWithImplementation.ts (1 errors) ====
     abstract class A {
         abstract foo() {}
-                       ~~
+                 ~~~
 !!! error TS1245: Method 'foo' cannot have an implementation because it is marked abstract.
     }

Original file line number	Diff line number	Diff line change
`@@ -11175,7 +11175,7 @@ namespace ts {`
`11175`	`11175`	`// Abstract methods cannot have an implementation.`
`11176`	`11176`	`// Extra checks are to avoid reporting multiple errors relating to the "abstractness" of the node.`
`11177`	`11177`	`if (node.flags & NodeFlags.Abstract && node.body) {`
`11178`		`- error(node.body, Diagnostics.Method_0_cannot_have_an_implementation_because_it_is_marked_abstract, declarationNameToString(node.name));`
	`11178`	`+ error(node, Diagnostics.Method_0_cannot_have_an_implementation_because_it_is_marked_abstract, declarationNameToString(node.name));`
`11179`	`11179`	`}`
`11180`	`11180`	`}`
`11181`	`11181`