Merge branch 'dev'

README.md

@@ -34,7 +34,7 @@ Timbre was built in a day after I finally lost my patience trying to configure L
 Depend on Timbre in your `project.clj`:
 
 ```clojure
-[com.taoensso/timbre "0.5.1"]
+[com.taoensso/timbre "0.5.2"]
 ```
 
 and `require` the library:

project.clj

@@ -1,4 +1,4 @@
-(defproject com.taoensso/timbre "0.5.1"
+(defproject com.taoensso/timbre "0.5.2"
   :description "Simple, flexible, all-Clojure logging. No XML!"
   :url "https://github.com/ptaoussanis/timbre"
   :license {:name "Eclipse Public License"}

src/timbre/core.clj

@@ -5,7 +5,7 @@
             [clj-stacktrace.repl :as stacktrace]
             [postal.core :as postal])
   (:import  [java.util Date Locale]
-            [java.text SimpleDateFormat]))
+            java.text.SimpleDateFormat))
 
 ;;;; Default configuration and appenders
 
@@ -71,13 +71,13 @@
           ;; {:from "me@draines.com" :to "foo@example.com"}
           :postal nil}}))
 
-(defn set-config! [[k & ks] val] (swap! config assoc-in (cons k ks) val))
-(defn set-level!  [level]  (set-config! [:current-level] level))
-
 ;;;; Define and sort logging levels
 
-(def ^:private ordered-levels [:trace :debug :info :warn :error :fatal])
+(def ^:private ordered-levels [:trace :debug :info :warn :error :fatal :report])
 (def ^:private scored-levels  (zipmap ordered-levels (range)))
+(defn check-level
+  [x] (when-not (some #{x} ordered-levels)
+        (throw (Exception. (str "Invalid logging level: " x)))))
 
 (def compare-levels
   (memoize (fn [x y] (- (scored-levels x) (scored-levels y)))))
@@ -85,6 +85,14 @@
 (defn sufficient-level?
   [level] (>= (compare-levels level (:current-level @config)) 0))
 
+;;;; Config helpers
+
+(defn set-config! [[k & ks] val] (swap! config assoc-in (cons k ks) val))
+(defn set-level!
+  [level]
+  (check-level level)
+  (set-config! [:current-level] level))
+
 ;;;; Appender-fn decoration
 
 (defn- make-timestamp-fn
@@ -197,8 +205,9 @@
 (defmacro log
   "Dispatches given arguments to relevant appender-fns iff logging level is
   sufficient."
-  {:arglists '([message & more] [throwable message & more])}
+  {:arglists '([level message & more] [level throwable message & more])}
   [level & args]
+  (check-level level)
   `(let [level# ~level]
      (when (sufficient-level? level#)
        (when-let [juxt-fn# (@juxt-cache level#)] ; Any relevant appenders?
@@ -226,7 +235,7 @@
          (log :trace "arg1"))
 
 (defmacro spy
-  "Evaluates expression and logs its form and result. Returns the result.
+  "Evaluates expression and logs its form and result. Always returns the result.
   Defaults to :debug logging level."
   ([expr] `(spy :debug ~expr))
   ([level expr]