Prevent warnings on unused/unhandled code paths

src/rebar_git_resource.erl

@@ -124,7 +124,7 @@ parse_git_url(not_scp, Url) ->
     case rebar_uri:parse(Url, UriOpts) of
         #{path := Path, host := Host} ->
             {ok, {Host, filename:rootname(Path, ".git")}};
-        {error, Reason} ->
+        {error, _, Reason} ->
             {error, Reason}
     end.
 
@@ -174,7 +174,7 @@ maybe_warn_local_url(Url) ->
               "Use remote git resources, or a plugin for local dependencies.",
     case parse_git_url(Url) of
         {error, no_scheme} -> ?WARN(WarnStr, [Url]);
-        {error, {no_default_port, _, _}} -> ?WARN(WarnStr, [Url]);
+        {error, no_default_port} -> ?WARN(WarnStr, [Url]);
         {error, {malformed_url, _, _}} -> ?WARN(WarnStr, [Url]);
         _ -> ok
     end.

src/rebar_uri.erl

@@ -17,12 +17,13 @@ parse(URIString) ->
 parse(URIString, URIOpts) ->
     case uri_string:parse(URIString) of
         #{path := ""} = Map -> apply_opts(Map#{path => "/"}, URIOpts);
-        Map -> apply_opts(Map, URIOpts)
+        Map when is_map(Map) -> apply_opts(Map, URIOpts);
+        {error, _, _} = E -> E
     end.
 -else.
 -spec parse(URIString) -> URIMap when
     URIString :: iodata(),
-    URIMap :: map() | {error, atom(), term()}.
+    URIMap :: map() | {error, term(), term()}.
 
 parse(URIString) ->
     parse(URIString, []).
@@ -33,7 +34,7 @@ parse(URIString, URIOpts) ->
             %% no additional parser/term info available to us,
             %% e.g. see what uri_string returns in
             %% uri_string:parse(<<"h$ttp:::://////lolz">>).
-            {error, Reason, ""};
+            {error, "", Reason};
         {ok, {Scheme, UserInfo, Host, Port, Path, Query}} ->
             #{
                 scheme => rebar_utils:to_list(Scheme),