refactor: remove unnecessary branches

kgryte · kgryte · commit 8ea4115e5c1d · 2023-06-28T18:08:51.000-07:00
diff --git a/lib/node_modules/@stdlib/string/first/lib/main.js b/lib/node_modules/@stdlib/string/first/lib/main.js
@@ -66,16 +66,10 @@ function first( str, n ) {
 	if ( !isString( str ) ) {
 		throw new TypeError( format( 'invalid argument. First argument must be a string. Value: `%s`.', str ) );
 	}
-	if ( str.length === 0 ) {
-		return '';
-	}
 	if ( arguments.length > 1 ) {
 		if ( !isNonNegativeInteger( n ) ) {
 			throw new TypeError( format( 'invalid argument. Second argument must be a nonnegative integer. Value: `%s`.', n ) );
 		}
-		if ( str.length <= n ) {
-			return str;
-		}
 		nextBreak = 0;
 		while ( n > 0 ) {
 			nextBreak = nextGraphemeClusterBreak( str, nextBreak );
@@ -84,7 +78,7 @@ function first( str, n ) {
 	} else {
 		nextBreak = nextGraphemeClusterBreak( str );
 	}
-	// If `nextBreak` equals -1, it means that `str` is a string with a single grapheme cluster. In that case, we return `str` itself:
+	// Value of `nextBreak` will be -1 if and only if `str` is an empty string or `str` has only 1 extended grapheme cluster...
 	if ( str === '' || nextBreak === -1 ) {
 		return str;
 	}
