metosin / malli

High-performance data-driven data specification library for Clojure/Script.
Eclipse Public License 2.0
1.5k stars 211 forks source link

mr/set-default-registry! takes varargs #809

Closed ikitommi closed 1 year ago

ikitommi commented 1 year ago

Not sure if this is a good idea, but this:

(mr/set-default-registry!
  ;; linear search
  (mr/composite-registry
    ;; immutable registry
    {:map (m/-map-schema)}
    ;; mutable (spec-like) registry
    (mr/mutable-registry registry)
    ;; on the perils of dynamic scope
    (mr/dynamic-registry)))

... could be written as:

;; linear search
(mr/set-default-registry!
 ;; immutable registry
 {:map (m/-map-schema)}
 ;; mutable (spec-like) registry
 (mr/mutable-registry registry)
 ;; on the perils of dynamic scope
 (mr/dynamic-registry))

comments welcome.

ikitommi commented 1 year ago

Maybe not. Saving few keystrokes.