Make --column imply --line-number.

Closes #243

doc/rg.1

@@ -183,6 +183,7 @@ Show column numbers (1 based) in output.
 This only shows the column numbers for the first match on each line.
 Note that this doesn\[aq]t try to account for Unicode.
 One byte is equal to one column.
+This implies \-\-line\-number.
 .RS
 .RE
 .TP

doc/rg.1.md

@@ -126,7 +126,8 @@ Project home page: https://github.com/BurntSushi/ripgrep
 --column
 : Show column numbers (1 based) in output. This only shows the column
   numbers for the first match on each line. Note that this doesn't try
-  to account for Unicode. One byte is equal to one column.
+  to account for Unicode. One byte is equal to one column. This implies
+  --line-number.
 
 --context-separator *SEPARATOR*
 : The string to use when separating non-continuous context lines. Escape

src/app.rs

@@ -315,7 +315,8 @@ lazy_static! {
              "Show column numbers",
              "Show column numbers (1-based). This only shows the column \
               numbers for the first match on each line. This does not try \
-              to account for Unicode. One byte is equal to one column.");
+              to account for Unicode. One byte is equal to one column. This \
+              implies --line-number.");
         doc!(h, "context-separator",
              "Set the context separator string. [default: --]",
              "The string used to separate non-contiguous context lines in the \

src/args.rs

@@ -583,6 +583,7 @@ impl<'a> ArgMatches<'a> {
             false
         } else {
             self.is_present("line-number")
+            || self.is_present("column")
             || atty::on_stdout()
             || self.is_present("pretty")
             || self.is_present("vimgrep")

tests/tests.rs

@@ -107,8 +107,8 @@ sherlock!(columns, |wd: WorkDir, mut cmd: Command| {
     cmd.arg("--column");
     let lines: String = wd.stdout(&mut cmd);
     let expected = "\
-57:For the Doctor Watsons of this world, as opposed to the Sherlock
-49:be, to a very large extent, the result of luck. Sherlock Holmes
+1:57:For the Doctor Watsons of this world, as opposed to the Sherlock
+3:49:be, to a very large extent, the result of luck. Sherlock Holmes
 ";
     assert_eq!(lines, expected);
 });
@@ -781,7 +781,7 @@ clean!(regression_105_part2, "test", ".", |wd: WorkDir, mut cmd: Command| {
     cmd.arg("--column");
 
     let lines: String = wd.stdout(&mut cmd);
-    assert_eq!(lines, "foo:3:zztest\n");
+    assert_eq!(lines, "foo:1:3:zztest\n");
 });
 
 // See: https://github.com/BurntSushi/ripgrep/issues/127
@@ -1214,6 +1214,15 @@ clean!(feature_159_zero_max, "test", ".", |wd: WorkDir, mut cmd: Command| {
     wd.assert_err(&mut cmd);
 });
 
+// See: https://github.com/BurntSushi/ripgrep/issues/243
+clean!(feature_243_column_line, "test", ".", |wd: WorkDir, mut cmd: Command| {
+    wd.create("foo", "test");
+    cmd.arg("--column");
+
+    let lines: String = wd.stdout(&mut cmd);
+    assert_eq!(lines, "foo:1:1:test\n");
+});
+
 // See: https://github.com/BurntSushi/ripgrep/issues/263
 clean!(feature_263_sort_files, "test", ".", |wd: WorkDir, mut cmd: Command| {
     wd.create("foo", "test");