bumped elixir version and added formatter

Author: Kenneth Cross

.formatter.exs

@@ -0,0 +1 @@
+[inputs: ["mix.exs", "{web,config,lib,test}/**/*.{ex,exs}"]]

lib/briefly.ex

@@ -9,20 +9,20 @@ defmodule Briefly do
   end
 
   @type create_opts :: [
-    {:prefix, binary},
-    {:extname, binary},
-    {:directory, boolean}
-  ]
+          {:prefix, binary},
+          {:extname, binary},
+          {:directory, boolean}
+        ]
 
   @doc """
   Requests a temporary file to be created with the given options
   """
   @spec create(create_opts) ::
-    {:ok, binary} |
-    {:too_many_attempts, binary, pos_integer} |
-    {:no_tmp, [binary]}
+          {:ok, binary}
+          | {:too_many_attempts, binary, pos_integer}
+          | {:no_tmp, [binary]}
   def create(opts \\ []) do
-    GenServer.call(Briefly.Entry.server, {:create, opts})
+    GenServer.call(Briefly.Entry.server(), {:create, opts})
   end
 
   @doc """
@@ -34,8 +34,10 @@ defmodule Briefly do
     case create(opts) do
       {:ok, path} ->
         path
+
       {:too_many_attempts, tmp, attempts} ->
         raise "tried #{attempts} times to create a temporary file at #{tmp} but failed. What gives?"
+
       {:no_tmp, _tmps} ->
         raise "could not create a tmp directory to store temporary files. Set the :briefly :directory application setting to a directory with write permission"
     end
@@ -47,6 +49,6 @@ defmodule Briefly do
   """
   @spec cleanup :: [binary]
   def cleanup do
-    GenServer.call(Briefly.Entry.server, :cleanup)
+    GenServer.call(Briefly.Entry.server(), :cleanup)
   end
 end

lib/briefly/config.ex

@@ -9,13 +9,13 @@ defmodule Briefly.Config do
 
   defp get(key) do
     Application.get_env(:briefly, key, [])
-    |> List.wrap
+    |> List.wrap()
     |> Enum.find_value(&runtime_value/1)
   end
 
   defp runtime_value({:system, env_key}) do
     System.get_env(env_key)
   end
-  defp runtime_value(value), do: value
 
+  defp runtime_value(value), do: value
 end

lib/briefly/entry.ex

@@ -9,25 +9,27 @@ defmodule Briefly.Entry do
   use GenServer
 
   def start_link() do
-    GenServer.start_link(__MODULE__, :ok, [name: __MODULE__])
+    GenServer.start_link(__MODULE__, :ok, name: __MODULE__)
   end
 
   ## Callbacks
 
   @max_attempts 10
 
   def init(:ok) do
-    tmp = Briefly.Config.directory
-    cwd = Path.join(File.cwd!, "tmp")
+    tmp = Briefly.Config.directory()
+    cwd = Path.join(File.cwd!(), "tmp")
     ets = :ets.new(:briefly, [:private])
     {:ok, {[tmp, cwd], ets}}
   end
 
   def handle_call({:create, opts}, {pid, _ref}, {tmps, ets} = state) do
     options = opts |> Enum.into(%{})
+
     case find_tmp_dir(pid, tmps, ets) do
       {:ok, tmp, paths} ->
         {:reply, open(options, tmp, 0, pid, ets, paths), state}
+
       {:no_tmp, _} = error ->
         {:reply, error, state}
     end
@@ -53,6 +55,7 @@ defmodule Briefly.Entry do
     case :ets.lookup(ets, pid) do
       [{^pid, tmp, paths}] ->
         {:ok, tmp, paths}
+
       [] ->
         if tmp = ensure_tmp_dir(tmps) do
           :erlang.monitor(:process, pid)
@@ -65,7 +68,7 @@ defmodule Briefly.Entry do
   end
 
   defp ensure_tmp_dir(tmps) do
-    {mega, _, _} = :os.timestamp
+    {mega, _, _} = :os.timestamp()
     subdir = "/briefly-" <> i(mega)
     Enum.find_value(tmps, &write_tmp_dir(&1 <> subdir))
   end
@@ -77,13 +80,15 @@ defmodule Briefly.Entry do
     end
   end
 
-  defp open(%{directory: true} = options, tmp, attempts, pid, ets, paths) when attempts < @max_attempts do
+  defp open(%{directory: true} = options, tmp, attempts, pid, ets, paths)
+       when attempts < @max_attempts do
     path = path(options, tmp)
 
     case File.mkdir_p(path) do
       :ok ->
-        :ets.update_element(ets, pid, {3, [path|paths]})
+        :ets.update_element(ets, pid, {3, [path | paths]})
         {:ok, path}
+
       {:error, reason} when reason in [:eexist, :eacces] ->
         open(options, tmp, attempts + 1, pid, ets, paths)
     end
@@ -94,8 +99,9 @@ defmodule Briefly.Entry do
 
     case :file.write_file(path, "", [:write, :raw, :exclusive, :binary]) do
       :ok ->
-        :ets.update_element(ets, pid, {3, [path|paths]})
+        :ets.update_element(ets, pid, {3, [path | paths]})
         {:ok, path}
+
       {:error, reason} when reason in [:eexist, :eacces] ->
         open(options, tmp, attempts + 1, pid, ets, paths)
     end
@@ -109,27 +115,38 @@ defmodule Briefly.Entry do
   defp i(integer), do: Integer.to_string(integer)
 
   defp path(options, tmp) do
-    {_mega, sec, micro} = :os.timestamp
+    {_mega, sec, micro} = :os.timestamp()
     scheduler_id = :erlang.system_info(:scheduler_id)
-    IO.iodata_to_binary([tmp, "/",
-                         prefix(options),
-                         "-", i(sec), "-", i(micro), "-", i(scheduler_id),
-                         extname(options)])
+
+    IO.iodata_to_binary([
+      tmp,
+      "/",
+      prefix(options),
+      "-",
+      i(sec),
+      "-",
+      i(micro),
+      "-",
+      i(scheduler_id),
+      extname(options)
+    ])
   end
 
   defp prefix(%{prefix: value}), do: value
-  defp prefix(_), do: Briefly.Config.default_prefix
+  defp prefix(_), do: Briefly.Config.default_prefix()
 
   defp extname(%{extname: value}), do: value
-  defp extname(_), do: Briefly.Config.default_extname
+  defp extname(_), do: Briefly.Config.default_extname()
 
   defp cleanup(ets, pid) do
     case :ets.lookup(ets, pid) do
       [{^pid, _tmp, paths}] ->
         :ets.delete(ets, pid)
-        Enum.each paths, &File.rm_rf/1
+        Enum.each(paths, &File.rm_rf/1)
         paths
-      [] -> []
+
+      [] ->
+        []
     end
   end
 end

lib/briefly/supervisor.ex

@@ -11,6 +11,7 @@ defmodule Briefly.Supervisor do
     children = [
       worker(Briefly.Entry, [])
     ]
+
     supervise(children, strategy: :one_for_one)
   end
 end

mix.exs

@@ -2,23 +2,23 @@ defmodule Briefly.Mixfile do
   use Mix.Project
 
   def project do
-    [app: :briefly,
-     version: "0.3.1",
-     elixir: "~> 1.3",
-     source_url: "https://github.com/CargoSense/briefly",
-     build_embedded: Mix.env == :prod,
-     start_permanent: Mix.env == :prod,
-     package: package(),
-     deps: deps()]
+    [
+      app: :briefly,
+      version: "0.3.1",
+      elixir: "~> 1.6",
+      source_url: "https://github.com/CargoSense/briefly",
+      build_embedded: Mix.env() == :prod,
+      start_permanent: Mix.env() == :prod,
+      package: package(),
+      deps: deps()
+    ]
   end
 
   # Configuration for the OTP application
   #
   # Type `mix help compile.app` for more information
   def application do
-    [applications: [:logger],
-     mod: {Briefly, []},
-     env: default_env()]
+    [applications: [:logger], mod: {Briefly, []}, env: default_env()]
   end
 
   # Dependencies can be Hex packages:
@@ -32,22 +32,25 @@ defmodule Briefly.Mixfile do
   # Type `mix help deps` for more examples and options
   defp deps do
     [
-     {:ex_doc, "~> 0.8", only: :dev}
-   ]
+      {:ex_doc, "~> 0.8", only: :dev}
+    ]
   end
 
   defp package do
-    [description: "Simple, robust temporary file support",
-     files: ["lib", "config", "mix.exs", "README*", "LICENSE"],
-     contributors: ["Bruce Williams"],
-     licenses: ["Apache 2"],
-     links: %{github: "https://github.com/CargoSense/briefly"}]
+    [
+      description: "Simple, robust temporary file support",
+      files: ["lib", "config", "mix.exs", "README*", "LICENSE"],
+      contributors: ["Bruce Williams"],
+      licenses: ["Apache 2"],
+      links: %{github: "https://github.com/CargoSense/briefly"}
+    ]
   end
 
   defp default_env do
-    [directory: [{:system, "TMPDIR"}, {:system, "TMP"}, {:system, "TEMP"}, "/tmp"],
-     default_prefix: "briefly",
-     default_extname: ""]
+    [
+      directory: [{:system, "TMPDIR"}, {:system, "TMP"}, {:system, "TEMP"}, "/tmp"],
+      default_prefix: "briefly",
+      default_extname: ""
+    ]
   end
-
 end