From ae7bf03b488f5fdefbe5a89c68264236c747671e Mon Sep 17 00:00:00 2001
From: Erik Verbruggen <erik.verbruggen@digia.com>
Date: Wed, 17 Apr 2013 15:15:14 +0200
Subject: [PATCH] C++: Removed unused ObjC keyword highlighting.

Change-Id: Ie4db459ba9942d703ad3bc5a0f6805e39f729e9a
Reviewed-by: Nikolai Kosjar <nikolai.kosjar@digia.com>
---
 src/plugins/cppeditor/cppeditor.cpp    | 110 -------------------------
 src/plugins/cpptools/cppsemanticinfo.h |   1 -
 2 files changed, 111 deletions(-)

diff --git a/src/plugins/cppeditor/cppeditor.cpp b/src/plugins/cppeditor/cppeditor.cpp
index dace2102d9d..2c9868e4709 100644
--- a/src/plugins/cppeditor/cppeditor.cpp
+++ b/src/plugins/cppeditor/cppeditor.cpp
@@ -2113,16 +2113,8 @@ void CPPEditorWidget::updateSemanticInfo(const SemanticInfo &semanticInfo)
                 }
             }
         }
-
-#if 0 // ### TODO: enable objc semantic highlighting
-        setExtraSelections(ObjCSelection, createSelections(document(),
-                                                           semanticInfo.objcKeywords,
-                                                           m_keywordFormat));
-#endif
     }
 
-
-
     setExtraSelections(UnusedSymbolSelection, unusedSelections);
 
     if (! m_renameSelections.isEmpty())
@@ -2137,100 +2129,6 @@ void CPPEditorWidget::updateSemanticInfo(const SemanticInfo &semanticInfo)
     updateFunctionDeclDefLink();
 }
 
-namespace {
-
-class FindObjCKeywords: public ASTVisitor
-{
-public:
-    FindObjCKeywords(TranslationUnit *unit)
-        : ASTVisitor(unit)
-    {}
-
-    QList<SemanticInfo::Use> operator()()
-    {
-        _keywords.clear();
-        accept(translationUnit()->ast());
-        return _keywords;
-    }
-
-    virtual bool visit(ObjCClassDeclarationAST *ast)
-    {
-        addToken(ast->interface_token);
-        addToken(ast->implementation_token);
-        addToken(ast->end_token);
-        return true;
-    }
-
-    virtual bool visit(ObjCClassForwardDeclarationAST *ast)
-    { addToken(ast->class_token); return true; }
-
-    virtual bool visit(ObjCProtocolDeclarationAST *ast)
-    { addToken(ast->protocol_token); addToken(ast->end_token); return true; }
-
-    virtual bool visit(ObjCProtocolForwardDeclarationAST *ast)
-    { addToken(ast->protocol_token); return true; }
-
-    virtual bool visit(ObjCProtocolExpressionAST *ast)
-    { addToken(ast->protocol_token); return true; }
-
-    virtual bool visit(ObjCTypeNameAST *) { return true; }
-
-    virtual bool visit(ObjCEncodeExpressionAST *ast)
-    { addToken(ast->encode_token); return true; }
-
-    virtual bool visit(ObjCSelectorExpressionAST *ast)
-    { addToken(ast->selector_token); return true; }
-
-    virtual bool visit(ObjCVisibilityDeclarationAST *ast)
-    { addToken(ast->visibility_token); return true; }
-
-    virtual bool visit(ObjCPropertyAttributeAST *ast)
-    {
-        const Identifier *attrId = identifier(ast->attribute_identifier_token);
-        if (attrId == control()->objcAssignId()
-                || attrId == control()->objcCopyId()
-                || attrId == control()->objcGetterId()
-                || attrId == control()->objcNonatomicId()
-                || attrId == control()->objcReadonlyId()
-                || attrId == control()->objcReadwriteId()
-                || attrId == control()->objcRetainId()
-                || attrId == control()->objcSetterId())
-            addToken(ast->attribute_identifier_token);
-        return true;
-    }
-
-    virtual bool visit(ObjCPropertyDeclarationAST *ast)
-    { addToken(ast->property_token); return true; }
-
-    virtual bool visit(ObjCSynthesizedPropertiesDeclarationAST *ast)
-    { addToken(ast->synthesized_token); return true; }
-
-    virtual bool visit(ObjCDynamicPropertiesDeclarationAST *ast)
-    { addToken(ast->dynamic_token); return true; }
-
-    virtual bool visit(ObjCFastEnumerationAST *ast)
-    { addToken(ast->for_token); addToken(ast->in_token); return true; }
-
-    virtual bool visit(ObjCSynchronizedStatementAST *ast)
-    { addToken(ast->synchronized_token); return true; }
-
-protected:
-    void addToken(unsigned token)
-    {
-        if (token) {
-            SemanticInfo::Use use;
-            getTokenStartPosition(token, &use.line, &use.column);
-            use.length = tokenAt(token).length();
-            _keywords.append(use);
-        }
-    }
-
-private:
-    QList<SemanticInfo::Use> _keywords;
-};
-
-} // anonymous namespace
-
 SemanticHighlighter::Source CPPEditorWidget::currentSource(bool force)
 {
     int line = 0, column = 0;
@@ -2326,7 +2224,6 @@ SemanticInfo SemanticHighlighter::semanticInfo(const Source &source)
             && m_lastSemanticInfo.doc->fileName() == source.fileName) {
         semanticInfo.snapshot = m_lastSemanticInfo.snapshot; // ### TODO: use the new snapshot.
         semanticInfo.doc = m_lastSemanticInfo.doc;
-        semanticInfo.objcKeywords = m_lastSemanticInfo.objcKeywords;
     }
     m_mutex.unlock();
 
@@ -2337,13 +2234,6 @@ SemanticInfo SemanticHighlighter::semanticInfo(const Source &source)
             doc->control()->setTopLevelDeclarationProcessor(this);
             doc->check();
             semanticInfo.doc = doc;
-
-#if 0
-            if (TranslationUnit *unit = doc->translationUnit()) {
-                FindObjCKeywords findObjCKeywords(unit); // ### remove me
-                objcKeywords = findObjCKeywords();
-            }
-#endif
         }
     }
 
diff --git a/src/plugins/cpptools/cppsemanticinfo.h b/src/plugins/cpptools/cppsemanticinfo.h
index a6c0a7f416c..8cdc75b7909 100644
--- a/src/plugins/cpptools/cppsemanticinfo.h
+++ b/src/plugins/cpptools/cppsemanticinfo.h
@@ -67,7 +67,6 @@ public:
     CPlusPlus::Snapshot snapshot;
     CPlusPlus::Document::Ptr doc;
     LocalUseMap localUses;
-    QList<Use> objcKeywords;
 };
 
 } // namespace CppTools