linter: warn on BOM and deprecated stuff

nette · Oct 27, 2021 · aca81a7 · aca81a7
1 parent a209777
commit aca81a7
Showing 1 changed file with 20 additions and 3 deletions.
diff --git a/bin/latte-lint b/bin/latte-lint
@@ -84,16 +84,33 @@ function createLatte(): Latte\Engine
 
 function lintLatte(Latte\Engine $latte, string $file): bool
 {
+	set_error_handler(function (int $severity, string $message) use ($file) {
+		if ($severity === E_USER_DEPRECATED) {
+			fwrite(STDERR, "[DEPRECATED] $file    $message\n");
+			return null;
+		}
+		return false;
+	});
+
+	$s = file_get_contents($file);
+	if (substr($s, 0, 3) === "\xEF\xBB\xBF") {
+		fwrite(STDERR, "[WARNING]    $file    contains BOM\n");
+		$contents = substr($s, 3);
+	}
+
 	try {
-		$code = $latte->compile(file_get_contents($file));
+		$code = $latte->compile($s);
 
 	} catch (Latte\CompileException $e) {
-		fwrite(STDERR, "[ERROR]    $file    {$e->getMessage()}\n");
+		fwrite(STDERR, "[ERROR]      $file    {$e->getMessage()}\n");
 		return false;
+
+	} finally {
+		restore_error_handler();
 	}
 
 	if ($error = lintPHP($code)) {
-		fwrite(STDERR, "[ERROR]    $file    $error\n");
+		fwrite(STDERR, "[ERROR]      $file    $error\n");
 		return false;
 	}