Address codereview comments:

pkg/stack_trace/lib/src/frame.dart

 // Copyright (c) 2013, 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.
 
 library frame;
 
 
 import 'package:path/path.dart' as path;
 
 import 'trace.dart';
@@ skipping 96 matching lines @@
     var match = _vmFrame.firstMatch(frame);
     if (match == null) {
       throw new FormatException("Couldn't parse VM stack trace line '$frame'.");
     }
 
     // Get the pieces out of the regexp match. Function, URI and line should
     // always be found. The column is optional.
     var member = match[1].replaceAll("<anonymous closure>", "<fn>");
     var uri = Uri.parse(match[2]);
     var line = int.parse(match[3]);
-    var col = null;
-    var col_match = match[4];
-    if (col_match != null) {
-      col = int.parse(col_match);
+    var column = null;
+    var columnMatch = match[4];
+    if (columnMatch != null) {
+      col = int.parse(columnMatch);

[nweiz, 2013/09/24 21:45:32]

This still refers to "col", as does the reference in [new Frame] below.

     }
     return new Frame(uri, line, col, member);
   }
 
   /// Parses a string representation of a Chrome/V8 stack frame.
   factory Frame.parseV8(String frame) {
     var match = _v8Frame.firstMatch(frame);
     if (match == null) {
       throw new FormatException("Couldn't parse V8 stack trace line '$frame'.");
     }
@@ skipping 62 matching lines @@
 
     var line = match[2] == null ? null : int.parse(match[2]);
     var column = match[3] == null ? null : int.parse(match[3]);
     return new Frame(uri, line, column, match[4]);
   }
 
   Frame(this.uri, this.line, this.column, this.member);
 
   String toString() => '$location in $member';
 }