scanner: Improve check of "\u" escape sequence

diff --git a/tests/Makefile.am b/tests/Makefile.am
index 1bc7c42061aed27f1686467104014175406eecaf..8448e1002bf2570028af0035fd08877e842081d7 100644 (file)
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -570,6 +570,7 @@ TESTS = \
        scanner/preprocessor-unexpected.test \
        scanner/regex-escape-invalid.test \
        scanner/regex-escape-u-digit-length.test \
+       scanner/regex-escape-u-empty.test \
        scanner/regex-escape-x-digit-length.test \
        scanner/regex-escape-x-empty.test \
        scanner/regex-escape-x.vala \
@@ -578,6 +579,7 @@ TESTS = \
        scanner/regex-multiple-s.test \
        scanner/regex-multiple-x.test \
        scanner/string-escape-u-digit-length.test \
+       scanner/string-escape-u-empty.test \
        scanner/string-escape-x-digit-length.test \
        scanner/string-escape-x-empty.test \
        scanner/string-escape-x.vala \

diff --git a/tests/scanner/regex-escape-u-empty.test b/tests/scanner/regex-escape-u-empty.test
new file mode 100644 (file)
index 0000000..73f6a32
--- /dev/null
+++ b/tests/scanner/regex-escape-u-empty.test
@@ -0,0 +1,5 @@
+Invalid Code
+
+void main () {
+       Regex foo = /\u/s;
+}

diff --git a/tests/scanner/string-escape-u-empty.test b/tests/scanner/string-escape-u-empty.test
new file mode 100644 (file)
index 0000000..3faa4bb
--- /dev/null
+++ b/tests/scanner/string-escape-u-empty.test
@@ -0,0 +1,5 @@
+Invalid Code
+
+void main () {
+       string foo = "\u";
+}

diff --git a/vala/valageniescanner.vala b/vala/valageniescanner.vala
index e6a1631fdfb31326247aded1e07851be9e44fce5..b5ff25f6a79f89251294f93d1188cfd0a9ec157c 100644 (file)
--- a/vala/valageniescanner.vala
+++ b/vala/valageniescanner.vala
@@ -235,12 +235,14 @@ public class Vala.Genie.Scanner {
                                                        current++;
                                                        token_length_in_chars++;
                                                        int digit_length;
-                                                       for (digit_length = 0; digit_length < 4 && current < end && current[0].isxdigit (); digit_length++) {
+                                                       for (digit_length = 0; current < end && current[0].isxdigit (); digit_length++) {
                                                                current++;
                                                                token_length_in_chars++;
                                                        }
-                                                       if (digit_length != 4) {
-                                                               Report.error (get_source_reference (token_length_in_chars), "\\u requires four hex digits");
+                                                       if (digit_length < 1) {
+                                                               Report.error (get_source_reference (token_length_in_chars), "\\u used with no following hex digits");
+                                                       } else if (digit_length < 4) {
+                                                               Report.error (get_source_reference (token_length_in_chars), "incomplete universal character name");
                                                        }
                                                        break;
                                                case 'x':
@@ -763,12 +765,14 @@ public class Vala.Genie.Scanner {
                                                        current++;
                                                        token_length_in_chars++;
                                                        int digit_length;
-                                                       for (digit_length = 0; digit_length < 4 && current < end && current[0].isxdigit (); digit_length++) {
+                                                       for (digit_length = 0; current < end && current[0].isxdigit (); digit_length++) {
                                                                current++;
                                                                token_length_in_chars++;
                                                        }
-                                                       if (digit_length != 4) {
-                                                               Report.error (get_source_reference (token_length_in_chars), "\\u requires four hex digits");
+                                                       if (digit_length < 1) {
+                                                               Report.error (get_source_reference (token_length_in_chars), "\\u used with no following hex digits");
+                                                       } else if (digit_length < 4) {
+                                                               Report.error (get_source_reference (token_length_in_chars), "incomplete universal character name");
                                                        }
                                                        break;
                                                case 'x':
@@ -1341,12 +1345,14 @@ public class Vala.Genie.Scanner {
                                                        current++;
                                                        token_length_in_chars++;
                                                        int digit_length;
-                                                       for (digit_length = 0; digit_length < 4 && current < end && current[0].isxdigit (); digit_length++) {
+                                                       for (digit_length = 0; current < end && current[0].isxdigit (); digit_length++) {
                                                                current++;
                                                                token_length_in_chars++;
                                                        }
-                                                       if (digit_length != 4) {
-                                                               Report.error (get_source_reference (token_length_in_chars), "\\u requires four hex digits");
+                                                       if (digit_length < 1) {
+                                                               Report.error (get_source_reference (token_length_in_chars), "\\u used with no following hex digits");
+                                                       } else if (digit_length < 4) {
+                                                               Report.error (get_source_reference (token_length_in_chars), "incomplete universal character name");
                                                        }
                                                        break;
                                                case 'x':

diff --git a/vala/valascanner.vala b/vala/valascanner.vala
index 9153c4bc48458403ef5d7106caf0d9b3ce626b13..a2e76b4d55b35fe971b2db54105b1984fc9f7c8e 100644 (file)
--- a/vala/valascanner.vala
+++ b/vala/valascanner.vala
@@ -220,12 +220,14 @@ public class Vala.Scanner {
                                                        current++;
                                                        token_length_in_chars++;
                                                        int digit_length;
-                                                       for (digit_length = 0; digit_length < 4 && current < end && current[0].isxdigit (); digit_length++) {
+                                                       for (digit_length = 0; current < end && current[0].isxdigit (); digit_length++) {
                                                                current++;
                                                                token_length_in_chars++;
                                                        }
-                                                       if (digit_length != 4) {
-                                                               Report.error (get_source_reference (token_length_in_chars), "\\u requires four hex digits");
+                                                       if (digit_length < 1) {
+                                                               Report.error (get_source_reference (token_length_in_chars), "\\u used with no following hex digits");
+                                                       } else if (digit_length < 4) {
+                                                               Report.error (get_source_reference (token_length_in_chars), "incomplete universal character name");
                                                        }
                                                        break;
                                                case 'x':
@@ -740,12 +742,14 @@ public class Vala.Scanner {
                                                        current++;
                                                        token_length_in_chars++;
                                                        int digit_length;
-                                                       for (digit_length = 0; digit_length < 4 && current < end && current[0].isxdigit (); digit_length++) {
+                                                       for (digit_length = 0; current < end && current[0].isxdigit (); digit_length++) {
                                                                current++;
                                                                token_length_in_chars++;
                                                        }
-                                                       if (digit_length != 4) {
-                                                               Report.error (get_source_reference (token_length_in_chars), "\\u requires four hex digits");
+                                                       if (digit_length < 1) {
+                                                               Report.error (get_source_reference (token_length_in_chars), "\\u used with no following hex digits");
+                                                       } else if (digit_length < 4) {
+                                                               Report.error (get_source_reference (token_length_in_chars), "incomplete universal character name");
                                                        }
                                                        break;
                                                case 'x':
@@ -1183,12 +1187,14 @@ public class Vala.Scanner {
                                                        current++;
                                                        token_length_in_chars++;
                                                        int digit_length;
-                                                       for (digit_length = 0; digit_length < 4 && current < end && current[0].isxdigit (); digit_length++) {
+                                                       for (digit_length = 0; current < end && current[0].isxdigit (); digit_length++) {
                                                                current++;
                                                                token_length_in_chars++;
                                                        }
-                                                       if (digit_length != 4) {
-                                                               Report.error (get_source_reference (token_length_in_chars), "\\u requires four hex digits");
+                                                       if (digit_length < 1) {
+                                                               Report.error (get_source_reference (token_length_in_chars), "\\u used with no following hex digits");
+                                                       } else if (digit_length < 4) {
+                                                               Report.error (get_source_reference (token_length_in_chars), "incomplete universal character name");
                                                        }
                                                        break;
                                                case 'x':