Skip to content

Commit b939fe1

Browse files
committed
replace old calls to clojure.core/read-string with clojure.edn/read-string
1 parent 92e5e07 commit b939fe1

5 files changed

Lines changed: 12 additions & 8 deletions

File tree

bin/cljsc.clj

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,14 +6,15 @@
66
; the terms of this license.
77
; You must not remove this notice, or any other, from this software.
88

9-
(require '[cljs.closure :as closure])
9+
(require '[cljs.closure :as closure]
10+
'[clojure.edn :as edn])
1011

1112
(defn transform-cl-args
1213
[args]
1314
(let [source (first args)
1415
opts-string (apply str (interpose " " (rest args)))
1516
options (when (> (count opts-string) 1)
16-
(try (read-string opts-string)
17+
(try (edn/read-string opts-string)
1718
(catch Exception e
1819
(binding [*out* *err*]
1920
(println "Failed to parse command line args:" e)))))]

src/main/clojure/cljs/closure.clj

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717
[cljs.env :as env]
1818
[cljs.foreign.node :refer [package-json-entries node-file-seq->libs-spec*]]
1919
[cljs.js-deps :as deps]
20+
[clojure.edn :as edn]
2021
[clojure.java.io :as io]
2122
[clojure.java.shell :as sh]
2223
[clojure.reflect]
@@ -2231,7 +2232,7 @@
22312232
([]
22322233
(get-upstream-deps* (. (Thread/currentThread) (getContextClassLoader))))
22332234
([classloader]
2234-
(let [upstream-deps (map #(read-string (slurp %))
2235+
(let [upstream-deps (map #(edn/read-string (slurp %))
22352236
(enumeration-seq (. classloader (getResources "deps.cljs"))))]
22362237
(apply merge-with
22372238
(fn [a b]

src/main/clojure/cljs/repl/browser.clj

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -299,7 +299,7 @@
299299
(constrain-order order
300300
(fn []
301301
(binding [*out* (or (and repl (.get outs repl)) *out*)]
302-
(print (read-string content))
302+
(print (edn/read-string content))
303303
(.flush *out*))))
304304
(server/send-and-close conn 200 "ignore__"))
305305

@@ -322,7 +322,7 @@
322322
(fn [val] (deliver return-value val)))
323323
(let [ret @return-value]
324324
(try
325-
(read-string ret)
325+
(edn/read-string ret)
326326
(catch Exception e
327327
{:status :error
328328
:value (str "Could not read return value: " ret)})))))

src/main/clojure/cljs/repl/reflect.clj

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111
(:require [cljs.repl.server :as server]
1212
[cljs.analyzer :as analyzer]
1313
[cljs.compiler :as compiler]
14+
[clojure.edn :as edn]
1415
[clojure.string :as str]
1516
[clojure.pprint :as pprint]))
1617

@@ -48,7 +49,7 @@
4849
(defn- url-decode [encoded & [encoding]]
4950
(java.net.URLDecoder/decode encoded (or encoding "UTF-8")))
5051

51-
(def read-url-string (comp read-string url-decode))
52+
(def read-url-string (comp edn/read-string url-decode))
5253

5354
(defn parse-param
5455
"Parses the query parameter of a path of the form \"/reflect?var=foo\"

src/main/clojure/cljs/repl/server.clj

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,8 @@
88

99
(ns cljs.repl.server
1010
(:refer-clojure :exclude [loaded-libs])
11-
(:require [clojure.string :as str])
11+
(:require [clojure.string :as str]
12+
[clojure.edn :as edn])
1213
(:import java.io.BufferedReader
1314
java.io.InputStreamReader
1415
java.io.ByteArrayOutputStream
@@ -191,7 +192,7 @@
191192
handler))
192193
handlers)]
193194
(if (= :post (:method request))
194-
(handler (read-string (:content request)) conn opts )
195+
(handler (edn/read-string (:content request)) conn opts )
195196
(handler request conn opts))
196197
(send-404 conn (:path request))))
197198
(catch Throwable t

0 commit comments

Comments
 (0)