Skip to content

Commit

Permalink
omit parens when possible
Browse files Browse the repository at this point in the history
  • Loading branch information
borodean committed Oct 5, 2021
1 parent 28de379 commit ed55367
Show file tree
Hide file tree
Showing 6 changed files with 12 additions and 11 deletions.
4 changes: 2 additions & 2 deletions benchmarks/complex.js
Expand Up @@ -122,13 +122,13 @@ suite
picocolors.yellow(`${++index}`)
)
})
.on("cycle", (event) => {
.on("cycle", event => {
let prefix = event.target.name === "picocolors" ? "+ " : " "
let name = event.target.name.padEnd("kleur/colors ".length)
let hz = formatNumber(event.target.hz.toFixed(0)).padStart(10)
process.stdout.write(`${prefix}${name}${picocolors.bold(hz)} ops/sec\n`)
})
.on("error", (event) => {
.on("error", event => {
process.stderr.write(picocolors.red(event.target.error.toString()) + "\n")
process.exit(1)
})
Expand Down
2 changes: 1 addition & 1 deletion benchmarks/loading.js
Expand Up @@ -11,7 +11,7 @@ for (let i = 0; i < RUNS; i++) {
output
.trim()
.split("\n")
.forEach((line) => {
.forEach(line => {
let [name, result] = line.split(" ")
results[name] = (results[name] || 0) + parseFloat(result)
})
Expand Down
4 changes: 2 additions & 2 deletions benchmarks/simple.js
Expand Up @@ -57,13 +57,13 @@ suite
.add("picocolors", () => {
out = picocolors.red("Add plugin to use time limit")
})
.on("cycle", (event) => {
.on("cycle", event => {
let prefix = event.target.name === "picocolors" ? "+ " : " "
let name = event.target.name.padEnd("kleur/colors ".length)
let hz = formatNumber(event.target.hz.toFixed(0)).padStart(10)
process.stdout.write(`${prefix}${name}${picocolors.bold(hz)} ops/sec\n`)
})
.on("error", (event) => {
.on("error", event => {
process.stderr.write(picocolors.red(event.target.error.toString()) + "\n")
process.exit(1)
})
Expand Down
6 changes: 3 additions & 3 deletions benchmarks/size.js
Expand Up @@ -5,10 +5,10 @@ let { get } = require("https")
let { bold, gray } = require("../picocolors.js")

async function getJSON(url) {
return new Promise((resolve) => {
get(url, (res) => {
return new Promise(resolve => {
get(url, res => {
let text = ""
res.on("data", (chunk) => {
res.on("data", chunk => {
text += chunk
})
res.on("end", () => {
Expand Down
3 changes: 2 additions & 1 deletion package.json
Expand Up @@ -40,7 +40,8 @@
"printWidth": 100,
"useTabs": true,
"tabWidth": 2,
"semi": false
"semi": false,
"arrowParens": "avoid"
},
"clean-publish": {
"cleanDocs": true
Expand Down
4 changes: 2 additions & 2 deletions picocolors.js
Expand Up @@ -10,7 +10,7 @@ let isColorSupported =

let formatter =
(open, close, replace = open) =>
(input) => {
input => {
let string = "" + input
let index = string.indexOf(close, open.length)
return ~index
Expand All @@ -27,7 +27,7 @@ let replaceClose = (string, close, replace, index) => {

let createColors = (enabled = isColorSupported) => ({
isColorSupported: enabled,
reset: enabled ? (s) => `\x1b[0m${s}\x1b[0m` : String,
reset: enabled ? s => `\x1b[0m${s}\x1b[0m` : String,
bold: enabled ? formatter("\x1b[1m", "\x1b[22m", "\x1b[22m\x1b[1m") : String,
dim: enabled ? formatter("\x1b[2m", "\x1b[22m", "\x1b[22m\x1b[2m") : String,
italic: enabled ? formatter("\x1b[3m", "\x1b[23m") : String,
Expand Down

0 comments on commit ed55367

Please sign in to comment.