Refactor utimes

c-vetter · Feb 17, 2017 · c9c62d8 · c9c62d8
1 parent a061b25
commit c9c62d8
Showing 1 changed file with 22 additions and 20 deletions.
diff --git a/lib/util/utimes.js b/lib/util/utimes.js
@@ -1,36 +1,38 @@
-var fs = require('graceful-fs')
-var path = require('path')
-var os = require('os')
+'use strict'
+
+const fs = require('graceful-fs')
+const os = require('os')
+const path = require('path')
 
 // HFS, ext{2,3}, FAT do not, Node.js v0.10 does not
 function hasMillisResSync () {
-  var tmpfile = path.join('millis-test-sync' + Date.now().toString() + Math.random().toString().slice(2))
+  let tmpfile = path.join('millis-test-sync' + Date.now().toString() + Math.random().toString().slice(2))
   tmpfile = path.join(os.tmpdir(), tmpfile)
 
   // 550 millis past UNIX epoch
-  var d = new Date(1435410243862)
+  const d = new Date(1435410243862)
   fs.writeFileSync(tmpfile, 'https://github.com/jprichardson/node-fs-extra/pull/141')
-  var fd = fs.openSync(tmpfile, 'r+')
+  const fd = fs.openSync(tmpfile, 'r+')
   fs.futimesSync(fd, d, d)
   fs.closeSync(fd)
   return fs.statSync(tmpfile).mtime > 1435410243000
 }
 
 function hasMillisRes (callback) {
-  var tmpfile = path.join('millis-test' + Date.now().toString() + Math.random().toString().slice(2))
+  let tmpfile = path.join('millis-test' + Date.now().toString() + Math.random().toString().slice(2))
   tmpfile = path.join(os.tmpdir(), tmpfile)
 
   // 550 millis past UNIX epoch
-  var d = new Date(1435410243862)
-  fs.writeFile(tmpfile, 'https://github.com/jprichardson/node-fs-extra/pull/141', function (err) {
+  const d = new Date(1435410243862)
+  fs.writeFile(tmpfile, 'https://github.com/jprichardson/node-fs-extra/pull/141', err => {
     if (err) return callback(err)
-    fs.open(tmpfile, 'r+', function (err, fd) {
+    fs.open(tmpfile, 'r+', (err, fd) => {
       if (err) return callback(err)
-      fs.futimes(fd, d, d, function (err) {
+      fs.futimes(fd, d, d, err => {
         if (err) return callback(err)
-        fs.close(fd, function (err) {
+        fs.close(fd, err => {
           if (err) return callback(err)
-          fs.stat(tmpfile, function (err, stats) {
+          fs.stat(tmpfile, (err, stats) => {
             if (err) return callback(err)
             callback(null, stats.mtime > 1435410243000)
           })
@@ -52,19 +54,19 @@ function timeRemoveMillis (timestamp) {
 
 function utimesMillis (path, atime, mtime, callback) {
   // if (!HAS_MILLIS_RES) return fs.utimes(path, atime, mtime, callback)
-  fs.open(path, 'r+', function (err, fd) {
+  fs.open(path, 'r+', (err, fd) => {
     if (err) return callback(err)
-    fs.futimes(fd, atime, mtime, function (futimesErr) {
-      fs.close(fd, function (closeErr) {
+    fs.futimes(fd, atime, mtime, futimesErr => {
+      fs.close(fd, closeErr => {
         if (callback) callback(futimesErr || closeErr)
       })
     })
   })
 }
 
 module.exports = {
-  hasMillisRes: hasMillisRes,
-  hasMillisResSync: hasMillisResSync,
-  timeRemoveMillis: timeRemoveMillis,
-  utimesMillis: utimesMillis
+  hasMillisRes,
+  hasMillisResSync,
+  timeRemoveMillis,
+  utimesMillis
 }