Refactor ItemReaderASTVisitor.

- Move imports handling to dedicated class ASTImportsHandler,
  since it has a reasonably large amount of code that interacts
  very little with the rest of the class.
- Move Properties item handling to dedicated class ASTPropertiesItemHandler
  for the same reason as above.
- Pass in a FileContext, replacing three other parameters.
- Remove the empty destructor.
- Remove the declaration of an unimplemented member function.
- Make the visit() functions private and mark them as overridden.
- Rename visitStatement() to better reflect its purpose.
- Implementation details:
    - Use auto where it makes sense.
    - Access private Item state only where it is necessary.
    - Coding style.

Change-Id: I221d296e375c881872898a96afff3dfc9f1a9a63
Reviewed-by: Joerg Bornemann <joerg.bornemann@theqtcompany.com>

1 files changed, 4 insertions, 0 deletions

diff --git a/src/lib/corelib/language/language.pri b/src/lib/corelib/language/language.pri
index 68c8139d7..e1f2fead0 100644
--- a/src/lib/corelib/language/language.pri
+++ b/src/lib/corelib/language/language.pri
@@ -2,6 +2,8 @@ include(../../../install_prefix.pri)
 
 HEADERS += \
     $$PWD/artifactproperties.h \
+    $$PWD/astimportshandler.h \
+    $$PWD/astpropertiesitemhandler.h \
     $$PWD/asttools.h \
     $$PWD/builtindeclarations.h \
     $$PWD/builtinvalue.h \
@@ -40,6 +42,8 @@ HEADERS += \
 
 SOURCES += \
     $$PWD/artifactproperties.cpp \
+    $$PWD/astimportshandler.cpp \
+    $$PWD/astpropertiesitemhandler.cpp \
     $$PWD/asttools.cpp \
     $$PWD/builtindeclarations.cpp \
     $$PWD/builtinvalue.cpp \