Fix autolink anchors for non URL safe function names

Author: padde

lib/ex_doc/formatter/html/autolink.ex

@@ -208,8 +208,10 @@ defmodule ExDoc.Formatter.HTML.Autolink do
     |> List.flatten()
     |> Enum.filter(&(&1 in locals))
     |> Enum.reduce(bin, fn (x, acc) ->
+         {prefix, _, function_name, arity} = split_function(x)
          escaped = Regex.escape(x)
-         Regex.replace(~r/(?<!\[)`(\s*(#{escaped})\s*)`(?!\])/, acc, "[`\\1`](#\\2)")
+         Regex.replace(~r/(?<!\[)`(\s*#{escaped}\s*)`(?!\])/, acc,
+           "[`#{function_name}/#{arity}`](##{prefix}#{enc_h function_name}/#{arity})")
        end)
   end
 
@@ -250,7 +252,7 @@ defmodule ExDoc.Formatter.HTML.Autolink do
          {prefix, mod_str, function_name, arity} = split_function(x)
          escaped = Regex.escape(x)
          Regex.replace(~r/(?<!\[)`(\s*#{escaped}\s*)`(?!\])/, acc,
-           "[`#{mod_str}.#{function_name}/#{arity}`](#{mod_str}.html##{prefix}#{function_name}/#{arity})")
+           "[`#{mod_str}.#{function_name}/#{arity}`](#{mod_str}.html##{prefix}#{enc_h function_name}/#{arity})")
        end)
   end
 
@@ -319,7 +321,7 @@ defmodule ExDoc.Formatter.HTML.Autolink do
          {_, mod_str, function_name, arity} = split_function(x)
          escaped = Regex.escape(x)
          Regex.replace(~r/(?<!\[)`(\s*:#{escaped}\s*)`(?!\])/, acc,
-           "[`\\1`](#{@erlang_docs}#{mod_str}.html##{function_name}-#{arity})")
+           "[`\\1`](#{@erlang_docs}#{mod_str}.html##{enc_h function_name}-#{arity})")
        end)
   end
 

mix.lock

@@ -2,8 +2,8 @@
   "excoveralls": {:hex, :excoveralls, "0.3.11"},
   "exjsx": {:hex, :exjsx, "3.2.0"},
   "hackney": {:hex, :hackney, "1.3.2"},
-  "hoedown": {:git, "git://github.com/hoedown/hoedown.git", "2a4cf17c70e6572ed42cdca3ea7ca3e768ed17be", []},
+  "hoedown": {:git, "https://github.com/hoedown/hoedown.git", "2a4cf17c70e6572ed42cdca3ea7ca3e768ed17be", []},
   "idna": {:hex, :idna, "1.0.2"},
   "jsx": {:hex, :jsx, "2.6.2"},
-  "markdown": {:git, "git://github.com/devinus/markdown.git", "cd0df79b6f1cc374499d47f6ba6aaab5096f874f", []},
+  "markdown": {:git, "https://github.com/devinus/markdown.git", "cd0df79b6f1cc374499d47f6ba6aaab5096f874f", []},
   "ssl_verify_hostname": {:hex, :ssl_verify_hostname, "1.0.5"}}

test/ex_doc/formatter/html/autolink_test.exs

@@ -10,7 +10,7 @@ defmodule ExDoc.Formatter.HTML.AutolinkTest do
     assert Autolink.local_doc("`example/2` then `example/2`",
       ["example/2"]) == "[`example/2`](#example/2) then [`example/2`](#example/2)"
     assert Autolink.local_doc("`  spaces/0  `", ["spaces/0"]) ==
-      "[`  spaces/0  `](#spaces/0)"
+      "[`spaces/0`](#spaces/0)"
     assert Autolink.local_doc("`example/1` and `example/2`",
       ["example/1", "example/2"]) == "[`example/1`](#example/1) and [`example/2`](#example/2)"
     assert Autolink.local_doc("`funny_name\?/1` and `funny_name!/2`",
@@ -33,6 +33,7 @@ defmodule ExDoc.Formatter.HTML.AutolinkTest do
     assert Autolink.local_doc("`!/1`", ["!/1"]) === "[`!/1`](#!/1)"
     assert Autolink.local_doc("`../2`", ["../2"]) === "[`../2`](#../2)"
     assert Autolink.local_doc("`--/2`", ["--/2"]) === "[`--/2`](#--/2)"
+    assert Autolink.local_doc("`<<>>/1`", ["<<>>/1"]) === "[`<<>>/1`](#%3C%3C%3E%3E/1)"
   end
 
   # project_functions