commit 7404d17ac76da876ae0b432d1fccf222a9e991fe
author Benjamin Peterson <benjamin@python.org> Thu Jul 02 04:32:59 2020 -0700
committer Copybara-Service <copybara-worker@google.com> Thu Jul 02 04:34:07 2020 -0700
tree e401b18d8ba1253a447a18bd99ffcc67d4061d6c
parent c01c1f143d5fbf8ae2402553905b29d1b278df12

Update build.proto docs for column numbers.

Starlark locations now include columns, but this was not reflected in the comments for various location fields.

Closes #11652.

PiperOrigin-RevId: 319390792

src/main/protobuf/build.proto

diff --git a/src/main/protobuf/build.proto b/src/main/protobuf/build.proto
index a245aa8..181a8e4 100644
--- a/src/main/protobuf/build.proto
+++ b/src/main/protobuf/build.proto
@@ -286,9 +286,10 @@
   required string rule_class = 2;
 
   // The BUILD file and line number of the location (formatted as
-  // <absolute_path>:<line_number>) in the rule's package's BUILD file where the
-  // rule instance was instantiated. The line number will be that of a rule
-  // invocation or macro call (that in turn invoked a rule). See
+  // <absolute_path>:<line_number>:<column_number>) in the rule's package's
+  // BUILD file where the rule instance was instantiated. The line number will
+  // be that of a rule invocation or macro call (that in turn invoked a
+  // rule). See
   // https://docs.bazel.build/versions/master/skylark/macros.html#macro-creation
   optional string location = 3;
 
@@ -373,8 +374,8 @@
   // The name of the source file (a label).
   required string name = 1;
 
-  // The location of the source file.  This is a path with line numbers, not
-  // a label in the build system.
+  // The location of the source file.  This is a path with a line number and a
+  // column number not a label in the build system.
   optional string location = 2;
 
   // Labels of .bzl (Starlark) files that are transitively loaded in this BUILD
@@ -411,7 +412,7 @@
   // The label of the target that generates the file.
   required string generating_rule = 2;
 
-  // The path of the output file (not a label).
+  // The path, line number, and column number of the output file (not a label).
   optional string location = 3;
 }