Skip to content

Commit 069fa57

Browse files
finalfantasiamarijnh
authored andcommitted
[clojure mode] Rename variables for clarity.
1 parent cf4a4a4 commit 069fa57

File tree

2 files changed

+10
-10
lines changed

2 files changed

+10
-10
lines changed

mode/clojure/clojure.js

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -137,7 +137,7 @@ CodeMirror.defineMode("clojure", function (options) {
137137
"with-local-vars", "with-meta", "with-open", "with-out-str",
138138
"with-precision", "with-redefs", "with-redefs-fn", "xml-seq", "zero?",
139139
"zipmap"];
140-
var formsThatHaveBodyParameter = [
140+
var haveBodyParameter = [
141141
"binding", "bound-fn", "case", "catch", "comment", "cond", "cond->",
142142
"cond->>", "condp", "def", "definterface", "defmethod", "defn",
143143
"defmacro", "defprotocol", "defrecord", "defstruct", "deftype", "do",
@@ -153,9 +153,9 @@ CodeMirror.defineMode("clojure", function (options) {
153153
var atom = createLookupMap(atoms);
154154
var specialForm = createLookupMap(specialForms);
155155
var coreSymbol = createLookupMap(coreSymbols);
156-
var hasBodyParameter = createLookupMap(formsThatHaveBodyParameter);
156+
var hasBodyParameter = createLookupMap(haveBodyParameter);
157157
var numberLiteral = /^[+\-]?\d+(?:(?:N|(?:[eE][+\-]?\d+))|(?:\.?\d*(?:M|(?:[eE][+\-]?\d+))?)|\/\d+|[xX][0-9a-fA-F]+|r[0-9a-zA-Z]+)?/;
158-
var symbol = /[!#'*+\-.\/:<>?_\w\xa1-\uffff]/;
158+
var symbolCharacter = /[!#'*+\-.\/:<>?_\w\xa1-\uffff]/;
159159

160160
function base(stream, state) {
161161
if (stream.eatSpace()) return ["space", null];
@@ -170,12 +170,12 @@ CodeMirror.defineMode("clojure", function (options) {
170170
if (ch === ";") {stream.skipToEnd(); return ["space", "comment"];}
171171
if (is(ch, /[#'@^`~]/)) return [null, "meta"];
172172

173-
var name = readSymbol(stream);
173+
var symbol = readSymbol(stream);
174174

175-
if (name === "comment" && state.lastToken === "(")
175+
if (symbol === "comment" && state.lastToken === "(")
176176
return (state.tokenize = inComment)(stream, state);
177-
if (is(name, atom) || name.charAt(0) === ":") return ["symbol", "atom"];
178-
if (is(name, specialForm) || is(name, coreSymbol)) return ["symbol", "keyword"];
177+
if (is(symbol, atom) || symbol.charAt(0) === ":") return ["symbol", "atom"];
178+
if (is(symbol, specialForm) || is(symbol, coreSymbol)) return ["symbol", "keyword"];
179179
if (state.lastToken === "(") return ["symbol", "builtin"]; // other operator
180180

181181
return ["symbol", "variable"];
@@ -214,7 +214,7 @@ CodeMirror.defineMode("clojure", function (options) {
214214

215215
while (ch = stream.next()) {
216216
if (ch === "\\") stream.next();
217-
else if (!is(ch, symbol)) {stream.backUp(1); break;}
217+
else if (!is(ch, symbolCharacter)) {stream.backUp(1); break;}
218218
}
219219

220220
return stream.current();

mode/clojure/test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -276,7 +276,7 @@
276276
" [atom nil][bracket }]"
277277
);
278278

279-
var formsThatHaveBodyParameter = [
279+
var haveBodyParameter = [
280280
"binding", "bound-fn", "case", "catch", "cond", "cond->",
281281
"cond->>", "condp", "def", "definterface", "defmethod", "defn",
282282
"defmacro", "defprotocol", "defrecord", "defstruct", "deftype", "do",
@@ -296,7 +296,7 @@
296296
}
297297
}
298298

299-
testFormsThatHaveBodyParameter(formsThatHaveBodyParameter);
299+
testFormsThatHaveBodyParameter(haveBodyParameter);
300300

301301
MT("should indent body argument of `comment` by `options.indentUnit` spaces",
302302
"[bracket (][comment comment foo bar]",

0 commit comments

Comments
 (0)