From 1fade2d0a665c8fb8b6668640fb3988deb0b56d8 Mon Sep 17 00:00:00 2001
From: "Kyle E. Mitchell" <kyle@kemitchell.com>
Date: Wed, 19 Oct 2016 13:42:34 -0700
Subject: [PATCH] fs: clarify fs.link and fs.linkSync arguments

Updates the argument names `srcpath` and `dstpath` to match the more
descriptive `existingPath` and `newPath` in the documentation.

PR-URL: https://github.com/nodejs/node/pull/9145
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
---
 lib/fs.js | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/lib/fs.js b/lib/fs.js
index a49a7d2e86ccad..e63e2c85800196 100644
--- a/lib/fs.js
+++ b/lib/fs.js
@@ -1056,24 +1056,24 @@ fs.symlinkSync = function(target, path, type) {
                          type);
 };
 
-fs.link = function(srcpath, dstpath, callback) {
+fs.link = function(existingPath, newPath, callback) {
   callback = makeCallback(callback);
-  if (!nullCheck(srcpath, callback)) return;
-  if (!nullCheck(dstpath, callback)) return;
+  if (!nullCheck(existingPath, callback)) return;
+  if (!nullCheck(newPath, callback)) return;
 
   var req = new FSReqWrap();
   req.oncomplete = callback;
 
-  binding.link(pathModule._makeLong(srcpath),
-               pathModule._makeLong(dstpath),
+  binding.link(pathModule._makeLong(existingPath),
+               pathModule._makeLong(newPath),
                req);
 };
 
-fs.linkSync = function(srcpath, dstpath) {
-  nullCheck(srcpath);
-  nullCheck(dstpath);
-  return binding.link(pathModule._makeLong(srcpath),
-                      pathModule._makeLong(dstpath));
+fs.linkSync = function(existingPath, newPath) {
+  nullCheck(existingPath);
+  nullCheck(newPath);
+  return binding.link(pathModule._makeLong(existingPath),
+                      pathModule._makeLong(newPath));
 };
 
 fs.unlink = function(path, callback) {