diff options
author | Ali Mohammad Pur <ali.mpfard@gmail.com> | 2021-06-07 12:03:09 +0430 |
---|---|---|
committer | Ali Mohammad Pur <Ali.mpfard@gmail.com> | 2021-06-07 14:45:49 +0430 |
commit | 71b4433b0d83714852e882a83808a7d4bbeb6117 (patch) | |
tree | de087995b3c2751c0297e05a8f5a9c4fb808d9ca /Userland/Libraries/LibGUI/INISyntaxHighlighter.cpp | |
parent | 3bac14e19e6bf7df9019b71ff47aa34ab89e516f (diff) | |
download | serenity-71b4433b0d83714852e882a83808a7d4bbeb6117.zip |
LibWeb+LibSyntax: Implement nested syntax highlighters
And use them to highlight javascript in HTML source.
This commit also changes how TextDocumentSpan::data is interpreted,
as it used to be an opaque pointer, but everyone stuffed an enum value
inside it, which made the values not unique to each highlighter;
that field is now a u64 serial id.
The syntax highlighters don't need to change their ways of stuffing
token types into that field, but a highlighter that calls another
nested highlighter needs to register the nested types for use with
token pairs.
Diffstat (limited to 'Userland/Libraries/LibGUI/INISyntaxHighlighter.cpp')
-rw-r--r-- | Userland/Libraries/LibGUI/INISyntaxHighlighter.cpp | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/Userland/Libraries/LibGUI/INISyntaxHighlighter.cpp b/Userland/Libraries/LibGUI/INISyntaxHighlighter.cpp index dcfe00f8c3..5a72560803 100644 --- a/Userland/Libraries/LibGUI/INISyntaxHighlighter.cpp +++ b/Userland/Libraries/LibGUI/INISyntaxHighlighter.cpp @@ -32,9 +32,9 @@ static Syntax::TextStyle style_for_token_type(const Gfx::Palette& palette, IniTo } } -bool IniSyntaxHighlighter::is_identifier(void* token) const +bool IniSyntaxHighlighter::is_identifier(u64 token) const { - auto ini_token = static_cast<GUI::IniToken::Type>(reinterpret_cast<size_t>(token)); + auto ini_token = static_cast<GUI::IniToken::Type>(token); return ini_token == GUI::IniToken::Type::Name; } @@ -53,7 +53,7 @@ void IniSyntaxHighlighter::rehighlight(const Palette& palette) span.attributes.color = style.color; span.attributes.bold = style.bold; span.is_skippable = token.m_type == IniToken::Type::Whitespace; - span.data = reinterpret_cast<void*>(token.m_type); + span.data = static_cast<u64>(token.m_type); spans.append(span); } m_client->do_set_spans(move(spans)); @@ -64,18 +64,18 @@ void IniSyntaxHighlighter::rehighlight(const Palette& palette) m_client->do_update(); } -Vector<IniSyntaxHighlighter::MatchingTokenPair> IniSyntaxHighlighter::matching_token_pairs() const +Vector<IniSyntaxHighlighter::MatchingTokenPair> IniSyntaxHighlighter::matching_token_pairs_impl() const { static Vector<MatchingTokenPair> pairs; if (pairs.is_empty()) { - pairs.append({ reinterpret_cast<void*>(IniToken::Type::LeftBracket), reinterpret_cast<void*>(IniToken::Type::RightBracket) }); + pairs.append({ static_cast<u64>(IniToken::Type::LeftBracket), static_cast<u64>(IniToken::Type::RightBracket) }); } return pairs; } -bool IniSyntaxHighlighter::token_types_equal(void* token1, void* token2) const +bool IniSyntaxHighlighter::token_types_equal(u64 token1, u64 token2) const { - return static_cast<GUI::IniToken::Type>(reinterpret_cast<size_t>(token1)) == static_cast<GUI::IniToken::Type>(reinterpret_cast<size_t>(token2)); + return static_cast<GUI::IniToken::Type>(token1) == static_cast<GUI::IniToken::Type>(token2); } IniSyntaxHighlighter::~IniSyntaxHighlighter() |