Attachment 357814 Details for Bug 192931 – Patch

[patch] Patch

bug-192931-20181220084412.patch (text/plain), 2.68 KB, created by Chris Dumez on 2018-12-20 08:44:13 PST

(hide)

Description:

Filename:

MIME Type:

Creator: Chris Dumez

Created: 2018-12-20 08:44:13 PST

Size: 2.68 KB

patch

obsolete

>Subversion Revision: 239441
>diff --git a/Tools/ChangeLog b/Tools/ChangeLog
>index 070e9778b18f1c95bee76c4c8cc19ce739f845b2..2de41bdb0cf95b8ad349e6f4c90e6c6eb4cf7ae6 100644
>--- a/Tools/ChangeLog
>+++ b/Tools/ChangeLog
>@@ -1,3 +1,14 @@
>+2018-12-20  Chris Dumez  <cdumez@apple.com>
>+
>+        Add style script rule to check for uses of std::optional<>
>+        https://bugs.webkit.org/show_bug.cgi?id=192931
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * Scripts/webkitpy/style/checkers/cpp.py:
>+        (check_wtf_optional):
>+        (check_style):
>+
> 2018-12-20  Carlos Garcia Campos  <cgarcia@igalia.com>
> 
>         [GTK][WPE] Bump webkitgtk-test-fonts to 0.0.8
>diff --git a/Tools/Scripts/webkitpy/style/checkers/cpp.py b/Tools/Scripts/webkitpy/style/checkers/cpp.py
>index eca69fd53a34d0f308dade36cd544bad2a2d1fca..c04d4408b2aa5955fe505a55a052d0c2c1ced9ad 100644
>--- a/Tools/Scripts/webkitpy/style/checkers/cpp.py
>+++ b/Tools/Scripts/webkitpy/style/checkers/cpp.py
>@@ -2378,6 +2378,26 @@ def check_wtf_move(clean_lines, line_number, file_state, error):
>     error(line_number, 'runtime/wtf_move', 4, "Use 'WTFMove()' instead of 'std::move()'.")
> 
> 
>+def check_wtf_optional(clean_lines, line_number, file_state, error):
>+    """Looks for use of 'std::optional<>' which should be replaced with 'WTF::Optional<>'.
>+
>+    Args:
>+      clean_lines: A CleansedLines instance containing the file.
>+      line_number: The number of the line to check.
>+      file_state: A _FileState instance which maintains information about
>+                  the state of things in the file.
>+      error: The function to call with any errors found.
>+    """
>+
>+    line = clean_lines.elided[line_number]  # Get rid of comments and strings.
>+
>+    using_std_move = search(r'\bstd::optional\s*\<', line)
>+    if not using_std_move:
>+        return
>+
>+    error(line_number, 'runtime/wtf_optional', 4, "Use 'WTF::Optional<>' instead of 'std::optional<>'.")
>+
>+
> def check_ctype_functions(clean_lines, line_number, file_state, error):
>     """Looks for use of the standard functions in ctype.h and suggest they be replaced
>        by use of equivilent ones in <wtf/ASCIICType.h>?.
>@@ -2915,6 +2935,7 @@ def check_style(clean_lines, line_number, file_extension, class_state, file_stat
>     check_using_namespace(clean_lines, line_number, file_extension, error)
>     check_max_min_macros(clean_lines, line_number, file_state, error)
>     check_wtf_move(clean_lines, line_number, file_state, error)
>+    check_wtf_optional(clean_lines, line_number, file_state, error)
>     check_ctype_functions(clean_lines, line_number, file_state, error)
>     check_switch_indentation(clean_lines, line_number, error)
>     check_braces(clean_lines, line_number, file_state, error)

Actions: View | Formatted Diff | Diff

Attachments on bug 192931: 357814 | 357818 | 357838