Fixed failing 4_External_Functions_A01_t01 test, updated co19-runtime.status with triaged test bug …

runtime/vm/parser.cc

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 #include "vm/parser.h"
 
 #include "vm/bigint_operations.h"
 #include "vm/class_finalizer.h"
 #include "vm/compiler.h"
 #include "vm/compiler_stats.h"
@@ skipping 2264 matching lines @@
   } else if (CurrentToken() == Token::kARROW) {
     ConsumeToken();
     const intptr_t expr_pos = TokenPos();
     AstNode* expr = ParseExpr(kAllowConst, kConsumeCascades);
     ASSERT(expr != NULL);
     current_block_->statements->Add(new ReturnNode(expr_pos, expr));
   } else if (IsLiteral("native")) {
     ParseNativeFunctionBlock(&params, func);
   } else if (func.is_external()) {
     // Body of an external method contains a single throw.
+    const String& function_name = String::Handle(func.name());

[hausner, 2012/10/19 21:43:25]

This needs to be a ZoneHandle I think.

[hausner, 2012/10/19 21:43:25]

This needs to be a ZoneHandle I think.

[Tom Ball, 2012/10/19 22:06:42]

Done.

     current_block_->statements->Add(
-        new ThrowNode(TokenPos(),
-            new LiteralNode(TokenPos(),
-                String::ZoneHandle(
-                    Symbols::New("External implementation missing."))), NULL));
+        ThrowNoSuchMethodError(TokenPos(), function_name));
   } else {
     UnexpectedToken();
   }
   SequenceNode* body = CloseBlock();
   current_block_->statements->Add(body);
   innermost_function_ = saved_innermost_function.raw();
   return CloseBlock();
 }
 
 
@@ skipping 7458 matching lines @@
 void Parser::SkipQualIdent() {
   ASSERT(IsIdentifier());
   ConsumeToken();
   if (CurrentToken() == Token::kPERIOD) {
     ConsumeToken();  // Consume the kPERIOD token.
     ExpectIdentifier("identifier expected after '.'");
   }
 }
 
 }  // namespace dart