1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

path: assert path.join() arguments equally

Browse Source 
Re-use `assertPath()` when asserting path argument types in `join()`
as throughout the rest of the `path` module.

This also ensures the same error message generated for posix as for
win32.

PR-URL: https://github.com/nodejs/io.js/pull/2159
Reviewed-By: Roman Reiss <me@silverwind.io>
This commit is contained in:
Phillip Johnsen 2015-07-11 00:13:59 +02:00 committed by Roman Reiss
parent bd01603201
commit 2ba84606a6
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/path.js
View File

@ -477,9 +477,7 @@ posix.join = function() {
var path = '';
for (var i = 0; i < arguments.length; i++) {
var segment = arguments[i];
if (typeof segment !== 'string') {
throw new TypeError('Arguments to path.join must be strings');
}
assertPath(segment);
if (segment) {
if (!path) {
path += segment;