Properly handle column for 'in' in 'not in' operator

Closes #14681.

Author: josevalim

lib/elixir/src/elixir_parser.yrl

@@ -752,8 +752,9 @@ build_op({UOp, _, [Left]}, {_Kind, {Line, Column, _} = Location, 'in'}, Right) w
   {UOp, Meta, [{'in', Meta, [Left, Right]}]};
 
 build_op(Left, {_Kind, Location, 'not in'}, Right) ->
-  Meta = meta_from_location(Location),
-  {'not', Meta, [{'in', Meta, [Left, Right]}]};
+  NotMeta = meta_from_location(Location),
+  InMeta = meta_from_location(element(3, Location)),
+  {'not', NotMeta, [{'in', InMeta, [Left, Right]}]};
 
 build_op(Left, {_Kind, Location, Op}, Right) ->
   {Op, newlines_op(Location) ++ meta_from_location(Location), [Left, Right]}.

lib/elixir/src/elixir_tokenizer.erl

@@ -1670,8 +1670,8 @@ tokenize_keyword(Kind, Rest, Line, Column, Atom, Length, Scope, Tokens) ->
 
       _ ->
         case {Kind, Tokens} of
-          {in_op, [{unary_op, NotInfo, 'not'} | T]} ->
-            add_token_with_eol({in_op, NotInfo, 'not in'}, T);
+          {in_op, [{unary_op, {NotLine, NotColumn, _}, 'not'} | T]} ->
+            add_token_with_eol({in_op, {NotLine, NotColumn, {Line, Column, nil}}, 'not in'}, T);
 
           {_, _} ->
             add_token_with_eol({Kind, {Line, Column, previous_was_eol(Tokens)}, Atom}, Tokens)

lib/elixir/test/elixir/kernel/parser_test.exs

@@ -447,6 +447,22 @@ defmodule Kernel.ParserTest do
                {:ok, {:=, context, [List.to_string(nfd_abba), 1]}}
     end
 
+    test "not in" do
+      assert Code.string_to_quoted!("a not in b", columns: true) ==
+               {:not, [line: 1, column: 3],
+                [
+                  {:in, [line: 1, column: 7],
+                   [{:a, [line: 1, column: 1], nil}, {:b, [line: 1, column: 10], nil}]}
+                ]}
+
+      assert Code.string_to_quoted!("a not  in b", columns: true) ==
+               {:not, [line: 1, column: 3],
+                [
+                  {:in, [line: 1, column: 8],
+                   [{:a, [line: 1, column: 1], nil}, {:b, [line: 1, column: 11], nil}]}
+                ]}
+    end
+
     test "handles maps and structs" do
       assert Code.string_to_quoted("%{}", columns: true) ==
                {:ok, {:%{}, [line: 1, column: 1], []}}