refactor: use early returns

---
type: pre_commit_static_analysis_report
description: Results of running static analysis checks when committing changes.
report:
  - task: lint_filenames
    status: passed
  - task: lint_editorconfig
    status: passed
  - task: lint_markdown
    status: na
  - task: lint_package_json
    status: na
  - task: lint_repl_help
    status: na
  - task: lint_javascript_src
    status: passed
  - task: lint_javascript_cli
    status: na
  - task: lint_javascript_examples
    status: na
  - task: lint_javascript_tests
    status: na
  - task: lint_javascript_benchmarks
    status: na
  - task: lint_python
    status: na
  - task: lint_r
    status: na
  - task: lint_c_src
    status: na
  - task: lint_c_examples
    status: na
  - task: lint_c_benchmarks
    status: na
  - task: lint_c_tests_fixtures
    status: na
  - task: lint_shell
    status: na
  - task: lint_typescript_declarations
    status: na
  - task: lint_typescript_tests
    status: na
  - task: lint_license_headers
    status: passed
---

Author: kgryte

lib/node_modules/@stdlib/_tools/doctest/compare-values/lib/main.js

@@ -202,7 +202,9 @@ function checkForPlaceholders( actual, expected ) {
 	case '<RangeError>':
 		return isRangeError( actual );
 	default:
-		// Uknown type, let it slide...
+		// Unknown type, let it slide...
+
+		// FIXME: should we compare constructor names here at a minimum?
 		return true;
 	}
 }
@@ -385,28 +387,32 @@ function compareValues( actual, expected ) {
 		if ( !checkPrimitive( actual, expected ) ) {
 			return 'Displayed return value is `'+expected+'`, but expected `'+actual+'` instead';
 		}
+		return null;
 	}
-	else if ( isTypedArray( actual ) ) {
+	if ( isTypedArray( actual ) ) {
 		return checkTypedArrays( actual, expected );
 	}
 	if ( isArray( actual ) ) {
 		if ( !checkArray( actual, expected ) ) {
 			return 'Displayed return value is `'+expected+'`, but expected `'+createAnnotationValue( actual )+'` instead';
 		}
+		return null;
 	}
-	else if ( isRegExp( actual ) ) {
+	if ( isRegExp( actual ) ) {
 		actual = actual.toString();
 		if ( actual !== expected ) {
 			return 'Displayed return value is `'+expected+'`, but expected `'+actual+'` instead';
 		}
+		return null;
 	}
-	else if ( isFunction( actual ) ) {
+	if ( isFunction( actual ) ) {
 		actual = actual.toString();
 		if ( actual !== expected ) {
 			return 'Displayed return value is `'+expected+'`, but expected `'+actual+'` instead';
 		}
+		return null;
 	}
-	else if ( isObjectLike( actual ) ) {
+	if ( isObjectLike( actual ) ) {
 		if ( !contains( expected, '...' ) ) {
 			try {
 				expectedValue = parse( expected );
@@ -419,6 +425,7 @@ function compareValues( actual, expected ) {
 				return 'Displayed return value is `'+expected+'`, but expected `'+actualString+'` instead';
 			}
 		}
+		return null;
 	}
 	return null;
 }