minor refactor

royqh1979 · Dec 2, 2022 · dcc7011 · dcc7011
1 parent 7794299
commit dcc7011
Show file tree

Hide file tree

Showing 8 changed files with 79 additions and 79 deletions.
diff --git a/libs/qsynedit/qsynedit/highlighter/asm.cpp b/libs/qsynedit/qsynedit/highlighter/asm.cpp
@@ -308,7 +308,7 @@ QString ASMHighlighter::getToken() const
     return mLineString.mid(mTokenPos,mRun-mTokenPos);
 }
 
-PHighlighterAttribute ASMHighlighter::getTokenAttribute() const
+const PHighlighterAttribute &ASMHighlighter::getTokenAttribute() const
 {
     switch(mTokenID) {
     case TokenId::Comment:

diff --git a/libs/qsynedit/qsynedit/highlighter/asm.h b/libs/qsynedit/qsynedit/highlighter/asm.h
@@ -90,7 +90,7 @@ class ASMHighlighter : public Highlighter
     QString languageName() override;
     HighlighterLanguage language() override;
     QString getToken() const override;
-    PHighlighterAttribute getTokenAttribute() const override;
+    const PHighlighterAttribute &getTokenAttribute() const override;
     int getTokenPos() override;
     void next() override;
     void setLine(const QString &newLine, int lineNumber) override;

diff --git a/libs/qsynedit/qsynedit/highlighter/base.cpp b/libs/qsynedit/qsynedit/highlighter/base.cpp
@@ -52,32 +52,32 @@ const QSet<QChar>& Highlighter::wordBreakChars() const
     return mWordBreakChars;
 }
 
-PHighlighterAttribute Highlighter::identifierAttribute() const
+const PHighlighterAttribute& Highlighter::identifierAttribute() const
 {
     return mIdentifierAttribute;
 }
 
-PHighlighterAttribute Highlighter::keywordAttribute() const
+const PHighlighterAttribute &Highlighter::keywordAttribute() const
 {
     return mKeywordAttribute;
 }
 
-PHighlighterAttribute Highlighter::commentAttribute() const
+const PHighlighterAttribute &Highlighter::commentAttribute() const
 {
     return mCommentAttribute;
 }
 
-PHighlighterAttribute Highlighter::stringAttribute() const
+const PHighlighterAttribute& Highlighter::stringAttribute() const
 {
     return mStringAttribute;
 }
 
-PHighlighterAttribute Highlighter::whitespaceAttribute() const
+const PHighlighterAttribute& Highlighter::whitespaceAttribute() const
 {
     return mWhitespaceAttribute;
 }
 
-PHighlighterAttribute Highlighter::symbolAttribute() const
+const PHighlighterAttribute& Highlighter::symbolAttribute() const
 {
     return mSymbolAttribute;
 }
@@ -208,7 +208,7 @@ void HighlighterAttribute::setStyles(const FontStyles &styles)
     }
 }
 
-QColor HighlighterAttribute::foreground() const
+const QColor& HighlighterAttribute::foreground() const
 {
     return mForeground;
 }
@@ -218,7 +218,7 @@ void HighlighterAttribute::setForeground(const QColor &color)
     mForeground = color;
 }
 
-QColor HighlighterAttribute::background() const
+const QColor &HighlighterAttribute::background() const
 {
     return mBackground;
 }

diff --git a/libs/qsynedit/qsynedit/highlighter/base.h b/libs/qsynedit/qsynedit/highlighter/base.h
@@ -92,10 +92,10 @@ class HighlighterAttribute {
     FontStyles styles() const;
     void setStyles(const FontStyles &styles);
 
-    QColor foreground() const;
+    const QColor &foreground() const;
     void setForeground(const QColor &color);
 
-    QColor background() const;
+    const QColor &background() const;
     void setBackground(const QColor &background);
 
     TokenType tokenType() const;
@@ -119,17 +119,17 @@ class Highlighter {
 
     const QSet<QChar>& wordBreakChars() const;
 
-    PHighlighterAttribute identifierAttribute() const;
+    const PHighlighterAttribute& identifierAttribute() const;
 
-    PHighlighterAttribute keywordAttribute() const;
+    const PHighlighterAttribute& keywordAttribute() const;
 
-    PHighlighterAttribute commentAttribute() const;
+    const PHighlighterAttribute& commentAttribute() const;
 
-    PHighlighterAttribute stringAttribute() const;
+    const PHighlighterAttribute& stringAttribute() const;
 
-    PHighlighterAttribute whitespaceAttribute() const;
+    const PHighlighterAttribute& whitespaceAttribute() const;
 
-    PHighlighterAttribute symbolAttribute() const;
+    const PHighlighterAttribute& symbolAttribute() const;
 
     virtual bool isIdentChar(const QChar& ch) const;
 
@@ -139,7 +139,7 @@ class Highlighter {
     virtual bool eol() const = 0;
     virtual HighlighterState getState() const = 0;
     virtual QString getToken() const=0;
-    virtual PHighlighterAttribute getTokenAttribute() const=0;
+    virtual const PHighlighterAttribute &getTokenAttribute() const=0;
     virtual int getTokenPos() = 0;
     virtual bool isKeyword(const QString& word);
     virtual void next() = 0;

diff --git a/libs/qsynedit/qsynedit/highlighter/cpp.cpp b/libs/qsynedit/qsynedit/highlighter/cpp.cpp
@@ -190,72 +190,72 @@ CppHighlighter::CppHighlighter(): Highlighter()
     resetState();
 }
 
-PHighlighterAttribute CppHighlighter::asmAttribute() const
+const PHighlighterAttribute &CppHighlighter::asmAttribute() const
 {
     return mAsmAttribute;
 }
 
-PHighlighterAttribute CppHighlighter::preprocessorAttribute() const
+const PHighlighterAttribute &CppHighlighter::preprocessorAttribute() const
 {
     return mPreprocessorAttribute;
 }
 
-PHighlighterAttribute CppHighlighter::invalidAttribute() const
+const PHighlighterAttribute &CppHighlighter::invalidAttribute() const
 {
     return mInvalidAttribute;
 }
 
-PHighlighterAttribute CppHighlighter::numberAttribute() const
+const PHighlighterAttribute &CppHighlighter::numberAttribute() const
 {
     return mNumberAttribute;
 }
 
-PHighlighterAttribute CppHighlighter::floatAttribute() const
+const PHighlighterAttribute &CppHighlighter::floatAttribute() const
 {
     return mFloatAttribute;
 }
 
-PHighlighterAttribute CppHighlighter::hexAttribute() const
+const PHighlighterAttribute &CppHighlighter::hexAttribute() const
 {
     return mHexAttribute;
 }
 
-PHighlighterAttribute CppHighlighter::octAttribute() const
+const PHighlighterAttribute &CppHighlighter::octAttribute() const
 {
     return mOctAttribute;
 }
 
-PHighlighterAttribute CppHighlighter::stringEscapeSequenceAttribute() const
+const PHighlighterAttribute &CppHighlighter::stringEscapeSequenceAttribute() const
 {
     return mStringEscapeSequenceAttribute;
 }
 
-PHighlighterAttribute CppHighlighter::charAttribute() const
+const PHighlighterAttribute &CppHighlighter::charAttribute() const
 {
     return mCharAttribute;
 }
 
-PHighlighterAttribute CppHighlighter::variableAttribute() const
+const PHighlighterAttribute &CppHighlighter::variableAttribute() const
 {
     return mVariableAttribute;
 }
 
-PHighlighterAttribute CppHighlighter::functionAttribute() const
+const PHighlighterAttribute &CppHighlighter::functionAttribute() const
 {
     return mFunctionAttribute;
 }
 
-PHighlighterAttribute CppHighlighter::classAttribute() const
+const PHighlighterAttribute &CppHighlighter::classAttribute() const
 {
     return mClassAttribute;
 }
 
-PHighlighterAttribute CppHighlighter::globalVarAttribute() const
+const PHighlighterAttribute &CppHighlighter::globalVarAttribute() const
 {
     return mGlobalVarAttribute;
 }
 
-PHighlighterAttribute CppHighlighter::localVarAttribute() const
+const PHighlighterAttribute &CppHighlighter::localVarAttribute() const
 {
     return mLocalVarAttribute;
 }
@@ -1438,7 +1438,7 @@ QString CppHighlighter::getToken() const
     return mLine.mid(mTokenPos,mRun-mTokenPos);
 }
 
-PHighlighterAttribute CppHighlighter::getTokenAttribute() const
+const PHighlighterAttribute &CppHighlighter::getTokenAttribute() const
 {
     switch (mTokenId) {
     case TokenId::Asm:

diff --git a/libs/qsynedit/qsynedit/highlighter/cpp.h b/libs/qsynedit/qsynedit/highlighter/cpp.h
@@ -56,33 +56,33 @@ class CppHighlighter: public Highlighter
 public:
     explicit CppHighlighter();
 
-    PHighlighterAttribute asmAttribute() const;
+    const PHighlighterAttribute &asmAttribute() const;
 
-    PHighlighterAttribute preprocessorAttribute() const;
+    const PHighlighterAttribute &preprocessorAttribute() const;
 
-    PHighlighterAttribute invalidAttribute() const;
+    const PHighlighterAttribute &invalidAttribute() const;
 
-    PHighlighterAttribute numberAttribute() const;
+    const PHighlighterAttribute &numberAttribute() const;
 
-    PHighlighterAttribute floatAttribute() const;
+    const PHighlighterAttribute &floatAttribute() const;
 
-    PHighlighterAttribute hexAttribute() const;
+    const PHighlighterAttribute &hexAttribute() const;
 
-    PHighlighterAttribute octAttribute() const;
+    const PHighlighterAttribute &octAttribute() const;
 
-    PHighlighterAttribute stringEscapeSequenceAttribute() const;
+    const PHighlighterAttribute &stringEscapeSequenceAttribute() const;
 
-    PHighlighterAttribute charAttribute() const;
+    const PHighlighterAttribute &charAttribute() const;
 
-    PHighlighterAttribute variableAttribute() const;
+    const PHighlighterAttribute &variableAttribute() const;
 
-    PHighlighterAttribute functionAttribute() const;
+    const PHighlighterAttribute &functionAttribute() const;
 
-    PHighlighterAttribute classAttribute() const;
+    const PHighlighterAttribute &classAttribute() const;
 
-    PHighlighterAttribute globalVarAttribute() const;
+    const PHighlighterAttribute &globalVarAttribute() const;
 
-    PHighlighterAttribute localVarAttribute() const;
+    const PHighlighterAttribute &localVarAttribute() const;
 
     static const QSet<QString> Keywords;
 
@@ -174,7 +174,7 @@ class CppHighlighter: public Highlighter
     bool isLastLineStringNotFinished(int state) const override;
     bool eol() const override;
     QString getToken() const override;
-    PHighlighterAttribute getTokenAttribute() const override;
+    const PHighlighterAttribute &getTokenAttribute() const override;
     int getTokenPos() override;
     void next() override;
     void setLine(const QString &newLine, int lineNumber) override;

diff --git a/libs/qsynedit/qsynedit/highlighter/glsl.cpp b/libs/qsynedit/qsynedit/highlighter/glsl.cpp
@@ -131,72 +131,72 @@ GLSLHighlighter::GLSLHighlighter(): Highlighter()
     resetState();
 }
 
-PHighlighterAttribute GLSLHighlighter::asmAttribute() const
+const PHighlighterAttribute &GLSLHighlighter::asmAttribute() const
 {
     return mAsmAttribute;
 }
 
-PHighlighterAttribute GLSLHighlighter::preprocessorAttribute() const
+const PHighlighterAttribute &GLSLHighlighter::preprocessorAttribute() const
 {
     return mPreprocessorAttribute;
 }
 
-PHighlighterAttribute GLSLHighlighter::invalidAttribute() const
+const PHighlighterAttribute &GLSLHighlighter::invalidAttribute() const
 {
     return mInvalidAttribute;
 }
 
-PHighlighterAttribute GLSLHighlighter::numberAttribute() const
+const PHighlighterAttribute &GLSLHighlighter::numberAttribute() const
 {
     return mNumberAttribute;
 }
 
-PHighlighterAttribute GLSLHighlighter::floatAttribute() const
+const PHighlighterAttribute &GLSLHighlighter::floatAttribute() const
 {
     return mFloatAttribute;
 }
 
-PHighlighterAttribute GLSLHighlighter::hexAttribute() const
+const PHighlighterAttribute &GLSLHighlighter::hexAttribute() const
 {
     return mHexAttribute;
 }
 
-PHighlighterAttribute GLSLHighlighter::octAttribute() const
+const PHighlighterAttribute &GLSLHighlighter::octAttribute() const
 {
     return mOctAttribute;
 }
 
-PHighlighterAttribute GLSLHighlighter::stringEscapeSequenceAttribute() const
+const PHighlighterAttribute &GLSLHighlighter::stringEscapeSequenceAttribute() const
 {
     return mStringEscapeSequenceAttribute;
 }
 
-PHighlighterAttribute GLSLHighlighter::charAttribute() const
+const PHighlighterAttribute &GLSLHighlighter::charAttribute() const
 {
     return mCharAttribute;
 }
 
-PHighlighterAttribute GLSLHighlighter::variableAttribute() const
+const PHighlighterAttribute &GLSLHighlighter::variableAttribute() const
 {
     return mVariableAttribute;
 }
 
-PHighlighterAttribute GLSLHighlighter::functionAttribute() const
+const PHighlighterAttribute &GLSLHighlighter::functionAttribute() const
 {
     return mFunctionAttribute;
 }
 
-PHighlighterAttribute GLSLHighlighter::classAttribute() const
+const PHighlighterAttribute &GLSLHighlighter::classAttribute() const
 {
     return mClassAttribute;
 }
 
-PHighlighterAttribute GLSLHighlighter::globalVarAttribute() const
+const PHighlighterAttribute &GLSLHighlighter::globalVarAttribute() const
 {
     return mGlobalVarAttribute;
 }
 
-PHighlighterAttribute GLSLHighlighter::localVarAttribute() const
+const PHighlighterAttribute &GLSLHighlighter::localVarAttribute() const
 {
     return mLocalVarAttribute;
 }
@@ -1301,7 +1301,7 @@ QString GLSLHighlighter::getToken() const
     return mLineString.mid(mTokenPos,mRun-mTokenPos);
 }
 
-PHighlighterAttribute GLSLHighlighter::getTokenAttribute() const
+const PHighlighterAttribute &GLSLHighlighter::getTokenAttribute() const
 {
     switch (mTokenId) {
     case TokenId::Asm: