Fix handling of carriage return.

Changelog.md

@@ -7,6 +7,7 @@ Compiler Features:
 
 
 Bugfixes:
+ * Error Reporter: Fix handling of carriage return.
  * SMTChecker: Fix internal error on ``FixedBytes`` constant initialized with string literal.
  * SMTChecker: Fix internal error on calling public getter on a state variable of type array (possibly nested) of structs.
  * SMTChecker: Fix internal error on pushing to ``string`` casted to ``bytes``.

liblangutil/SourceReferenceExtractor.cpp

@@ -19,6 +19,7 @@
 #include <liblangutil/CharStream.h>
 #include <liblangutil/Exceptions.h>
 
+#include <algorithm>
 #include <cmath>
 #include <iomanip>
 
@@ -106,7 +107,7 @@ SourceReference SourceReferenceExtractor::extract(SourceLocation const* _locatio
 		interest,
 		isMultiline,
 		line,
-		start.column,
-		end.column,
+		min(start.column, static_cast<int>(line.length())),
+		min(end.column, static_cast<int>(line.length()))
 	};
 }

scripts/check_style.sh

@@ -6,6 +6,8 @@ ERROR_LOG="$(mktemp -t check_style_XXXXXX.log)"
 
 EXCLUDE_FILES=(
     "libsolutil/picosha2.h"
+    "test/cmdlineTests/strict_asm_only_cr/input.yul"
+    "test/cmdlineTests/strict_asm_only_cr/err"
     "test/libsolutil/UTF8.cpp"
     "test/libsolidity/syntaxTests/license/license_cr_endings.sol"
     "test/libsolidity/syntaxTests/license/license_crlf_endings.sol"

test/cmdlineTests/strict_asm_only_cr/args

@@ -0,0 +1 @@
+--strict-assembly

test/cmdlineTests/strict_asm_only_cr/err

@@ -0,0 +1,6 @@
+Warning: Yul is still experimental. Please use the output with care.
+Error: Expected keyword "object".
+ --> strict_asm_only_cr/input.yul:1:2:
+  |
+1 | 
+  | ^

test/cmdlineTests/strict_asm_only_cr/exit

@@ -0,0 +1 @@
+1

test/cmdlineTests/strict_asm_only_cr/input.yul

@@ -0,0 +1 @@
+