From 23c3644871df0968ede9806a9005357ea89e1c9f Mon Sep 17 00:00:00 2001
From: Roy Qu <royqh1979@gmail.com>
Date: Thu, 28 Mar 2024 18:25:36 +0800
Subject: [PATCH] - enhancement: Highlight words in the string/comments.

---
 NEWS.md                                 |   1 +
 RedPandaIDE/editor.cpp                  |  12 +-
 RedPandaIDE/settings.cpp                |   1 -
 libs/qsynedit/qsynedit/syntaxer/cpp.cpp | 150 ++++++++++--------------
 4 files changed, 69 insertions(+), 95 deletions(-)

diff --git a/NEWS.md b/NEWS.md
index df82397a..dc17f420 100644
--- a/NEWS.md
+++ b/NEWS.md
@@ -103,6 +103,7 @@ Red Panda C++ Version 2.27
   - change: Don't turn on the code format option "indent class" by default.
   - enhancement: Add compiler set by choose the executable.
   - fix: Compile info for project doesn't have name of the project executable.
+  - enhancement: Highlight words in the string/comments.
     
 Red Panda C++ Version 2.26
   - enhancement: Code suggestion for embedded std::vectors.
diff --git a/RedPandaIDE/editor.cpp b/RedPandaIDE/editor.cpp
index e5bd2633..c100ce73 100644
--- a/RedPandaIDE/editor.cpp
+++ b/RedPandaIDE/editor.cpp
@@ -1340,8 +1340,10 @@ void Editor::onPreparePaintHighlightToken(int line, int aChar, const QString &to
             }
         }
         if (((attr->tokenType() == QSynedit::TokenType::Identifier)
-                || (attr->tokenType() == QSynedit::TokenType::Keyword)
-                || (attr->tokenType() == QSynedit::TokenType::Preprocessor)
+             || (attr->tokenType() == QSynedit::TokenType::Keyword)
+             || (attr->tokenType() == QSynedit::TokenType::Preprocessor)
+             || (attr->tokenType() == QSynedit::TokenType::String)
+             || (attr->tokenType() == QSynedit::TokenType::Comment)
                 )
             && (token == mCurrentHighlightedWord)) {
             // occurrencies of the selected identifier
@@ -1881,8 +1883,10 @@ void Editor::onStatusChanged(QSynedit::StatusChanges changes)
             if (getTokenAttriAtRowCol(caretXY(), token,attri)
                 && (
                     (attri->tokenType()==QSynedit::TokenType::Identifier)
-                    || (attri->tokenType() == QSynedit::TokenType::Keyword)
-                    || (attri->tokenType() == QSynedit::TokenType::Preprocessor)
+                        || (attri->tokenType() == QSynedit::TokenType::Keyword)
+                        || (attri->tokenType() == QSynedit::TokenType::Preprocessor)
+                        || (attri->tokenType() == QSynedit::TokenType::String)
+                        || (attri->tokenType() == QSynedit::TokenType::Comment)
                     )) {
                 mCurrentHighlightedWord = token;
             } else {
diff --git a/RedPandaIDE/settings.cpp b/RedPandaIDE/settings.cpp
index bfb11581..7ba4aa56 100644
--- a/RedPandaIDE/settings.cpp
+++ b/RedPandaIDE/settings.cpp
@@ -3186,7 +3186,6 @@ bool Settings::CompilerSets::addSets(const QString &folder, const QString& c_pro
         if (set->binDirs().contains(folder) && extractFileName(set->CCompiler())==c_prog)
             return false;
     }
-    qDebug()<<folder<<c_prog;
     // Default, release profile
     PCompilerSet baseSet = addSet(folder,c_prog);
     if (!baseSet || baseSet->name().isEmpty())
diff --git a/libs/qsynedit/qsynedit/syntaxer/cpp.cpp b/libs/qsynedit/qsynedit/syntaxer/cpp.cpp
index 4956503c..8541925f 100644
--- a/libs/qsynedit/qsynedit/syntaxer/cpp.cpp
+++ b/libs/qsynedit/qsynedit/syntaxer/cpp.cpp
@@ -349,9 +349,19 @@ void CppSyntaxer::procCppStyleComment()
         return;
     }
     mTokenId = TokenId::Comment;
+    bool isWord = isIdentChar(mLine[mRun]);
     while (mRun<mLineSize) {
-        if (isSpaceChar(mLine[mRun]))
+        if (isSpaceChar(mLine[mRun])) {
             break;
+        } else {
+            if (isWord) {
+                if (!isIdentChar(mLine[mRun]))
+                    break;
+            } else {
+                if (isIdentChar(mLine[mRun]))
+                    break;
+            }
+        }
         mRun++;
     }
     if (mRun<mLineSize) {
@@ -364,48 +374,51 @@ void CppSyntaxer::procCppStyleComment()
 
 void CppSyntaxer::procDocstring()
 {
-    bool finishProcess = false;
     mTokenId = TokenId::Comment;
     if (mRun>=mLineSize) {
         procNull();
         return;
     }
+    bool isWord = isIdentChar(mLine[mRun]);
     while (mRun<mLineSize) {
-        switch(mLine[mRun].unicode()) {
-        case ' ':
-        case '\t':
-            return;
-        case '*':
+        if(isSpaceChar(mLine[mRun]))
+            break;
+        else if (mLine[mRun] == '*') {
             if (mRun+1<mLineSize && mLine[mRun+1] == '/') {
+                if (isWord)
+                    break;
                 mRun += 2;
                 mRange.state = RangeState::rsUnknown;
-                finishProcess = true;
-            } else
-                mRun+=1;
-            break;
-        default:
-            mRun+=1;
+                break;
+            }
+        } else {
+            if (isWord) {
+                if (!isIdentChar(mLine[mRun]))
+                    break;
+            } else {
+                if (isIdentChar(mLine[mRun]))
+                    break;
+            }
         }
-        if (finishProcess)
-            break;
+        mRun++;
     }
 }
 
 void CppSyntaxer::procAnsiCStyleComment()
 {
-    bool finishProcess = false;
     mTokenId = TokenId::Comment;
     if (mRun>=mLineSize) {
         procNull();
         return;
     }
+    bool isWord = isIdentChar(mLine[mRun]);
     while (mRun<mLineSize) {
-        switch(mLine[mRun].unicode()) {
-        case ' ':
-        case '\t':
-            return;
-        case '*':
+        if(isSpaceChar(mLine[mRun])) {
+            break;
+        } else if (mLine[mRun] == '*') {
             if (mRun+1<mLineSize && mLine[mRun+1] == '/') {
+                if (isWord)
+                    break;
                 mRun += 2;
                 if (mRange.state == RangeState::rsDirectiveComment &&
                             mRun<mLineSize && mLine[mRun]!='\r' && mLine[mRun]!='\n') {
@@ -413,15 +426,18 @@ void CppSyntaxer::procAnsiCStyleComment()
                 } else {
                     mRange.state = RangeState::rsUnknown;
                 }
-                finishProcess = true;
-            } else
-                mRun+=1;
-            break;
-        default:
-            mRun+=1;
+                break;
+            }
+        } else {
+            if (isWord) {
+                if (!isIdentChar(mLine[mRun]))
+                    break;
+            } else {
+                if (isIdentChar(mLine[mRun]))
+                    break;
+            }
         }
-        if (finishProcess)
-            break;
+        mRun++;
     }
 }
 
@@ -1126,9 +1142,7 @@ void CppSyntaxer::procSlash()
             if (mRun < mLineSize) {
                 if (mRange.state == RangeState::rsAnsiC && mLine[mRun] == '*' ) {
                     mRange.state = RangeState::rsDocstring;
-                    procDocstring();
-                } else
-                    procAnsiCStyleComment();
+                }
             }
             return;
         case '=':
@@ -1208,63 +1222,6 @@ void CppSyntaxer::procStar()
     }
 }
 
-//void CppSyntaxer::stringEndProc()
-//{
-//    mTokenId = TokenId::String;
-//    if (mRun>=mLineSize) {
-//        nullProc();
-//        return;
-//    }
-//    mRange.state = RangeState::rsUnknown;
-
-//    while (mRun<mLineSize) {
-//        if (mLine[mRun]=='"') {
-//            mRun += 1;
-//            break;
-//        } else if (isSpaceChar(mLine[mRun])) {
-//            mRange.state = RangeState::rsString;
-//            return;
-//        } else if (mLine[mRun]=='\\') {
-//            if (mRun == mLineSize-1) {
-//                mRun+=1;
-//                mRange.state = RangeState::rsMultiLineString;
-//                return;
-//            }
-//            if (mRun+1<mLineSize) {
-//                switch(mLine[mRun+1].unicode()) {
-//                case '\'':
-//                case '"':
-//                case '\\':
-//                case '?':
-//                case 'a':
-//                case 'b':
-//                case 'f':
-//                case 'n':
-//                case 'r':
-//                case 't':
-//                case 'v':
-//                case '0':
-//                case '1':
-//                case '2':
-//                case '3':
-//                case '4':
-//                case '5':
-//                case '6':
-//                case '7':
-//                case '8':
-//                case '9':
-//                case 'x':
-//                case 'u':
-//                case 'U':
-//                    mRange.state = RangeState::rsMultiLineStringEscapeSeq;
-//                    return;
-//                }
-//            }
-//        }
-//        mRun += 1;
-//    }
-//}
-
 void CppSyntaxer::procStringEscapeSeq()
 {
     mTokenId = TokenId::StringEscapeSeq;
@@ -1341,8 +1298,11 @@ void CppSyntaxer::procString()
         return;
     }
     mTokenId = TokenId::String;
+    bool isWord = isIdentChar(mLine[mRun]);
     while (mRun < mLineSize) {
         if (mLine[mRun]=='"') {
+            if (isWord)
+                break;
             mRun++;
             mRange.state = RangeState::rsUnknown;
             return;
@@ -1384,6 +1344,16 @@ void CppSyntaxer::procString()
                     return;
                 }
             }
+        } else {
+            if (isWord) {
+                if (!isIdentChar(mLine[mRun])) {
+                    break;
+                }
+            } else {
+                if (isIdentChar(mLine[mRun])) {
+                    break;
+                }
+            }
         }
         mRun+=1;
     }
@@ -1395,7 +1365,7 @@ void CppSyntaxer::procStringStart()
 {
     mTokenId = TokenId::String;
     mRange.state = RangeState::rsString;
-    mRun += 1;
+    mRun++; //skip \"
     if (mRun>=mLineSize) {
         mRange.state = RangeState::rsStringUnfinished;
         return;