[PATCH v2 13/13] Remove app_common and all references to it

Ryan Long ryan.long at oarcorp.com
Mon Aug 2 20:44:29 UTC 2021


- Removed the includes of app_common.h
- Removed app_common references in build infrastructure
- Removed app_common.cc and app_common.h
---
 tester/covoar/CoverageReaderQEMU.cc |  1 -
 tester/covoar/CoverageReaderTSIM.cc |  1 -
 tester/covoar/DesiredSymbols.cc     |  1 -
 tester/covoar/ExecutableInfo.cc     |  1 -
 tester/covoar/Explanations.cc       |  1 -
 tester/covoar/GcovFunctionData.cc   |  1 -
 tester/covoar/Makefile              |  6 +----
 tester/covoar/ObjdumpProcessor.cc   |  1 -
 tester/covoar/ReportsBase.cc        |  1 -
 tester/covoar/ReportsHtml.cc        |  1 -
 tester/covoar/ReportsText.cc        |  1 -
 tester/covoar/SymbolTable.cc        |  1 -
 tester/covoar/TraceConverter.cc     |  1 -
 tester/covoar/TraceReaderLogQEMU.cc |  1 -
 tester/covoar/TraceWriterQEMU.cc    |  1 -
 tester/covoar/app_common.cc         | 49 -------------------------------------
 tester/covoar/app_common.h          | 18 --------------
 tester/covoar/coverage_converter.cc |  1 -
 tester/covoar/covmerge.cc           |  1 -
 tester/covoar/covoar.cc             |  1 -
 tester/covoar/wscript               |  3 +--
 21 files changed, 2 insertions(+), 91 deletions(-)
 delete mode 100644 tester/covoar/app_common.cc
 delete mode 100644 tester/covoar/app_common.h

diff --git a/tester/covoar/CoverageReaderQEMU.cc b/tester/covoar/CoverageReaderQEMU.cc
index a3d9d02..a44097d 100644
--- a/tester/covoar/CoverageReaderQEMU.cc
+++ b/tester/covoar/CoverageReaderQEMU.cc
@@ -14,7 +14,6 @@
 
 #include <rld.h>
 
-#include "app_common.h"
 #include "CoverageReaderQEMU.h"
 #include "CoverageMap.h"
 #include "ExecutableInfo.h"
diff --git a/tester/covoar/CoverageReaderTSIM.cc b/tester/covoar/CoverageReaderTSIM.cc
index 32e3aa7..d4b037d 100644
--- a/tester/covoar/CoverageReaderTSIM.cc
+++ b/tester/covoar/CoverageReaderTSIM.cc
@@ -14,7 +14,6 @@
 
 #include <rld.h>
 
-#include "app_common.h"
 #include "CoverageReaderTSIM.h"
 #include "CoverageMap.h"
 #include "ExecutableInfo.h"
diff --git a/tester/covoar/DesiredSymbols.cc b/tester/covoar/DesiredSymbols.cc
index d76c5af..54776ee 100644
--- a/tester/covoar/DesiredSymbols.cc
+++ b/tester/covoar/DesiredSymbols.cc
@@ -23,7 +23,6 @@
 #include "rld-files.h"
 
 #include "DesiredSymbols.h"
-#include "app_common.h"
 #include "CoverageMap.h"
 
 namespace Coverage {
diff --git a/tester/covoar/ExecutableInfo.cc b/tester/covoar/ExecutableInfo.cc
index 328f970..418a7f4 100644
--- a/tester/covoar/ExecutableInfo.cc
+++ b/tester/covoar/ExecutableInfo.cc
@@ -11,7 +11,6 @@
 
 #include "ExecutableInfo.h"
 #include "ObjdumpProcessor.h"
-#include "app_common.h"
 #include "CoverageMap.h"
 #include "SymbolTable.h"
 
diff --git a/tester/covoar/Explanations.cc b/tester/covoar/Explanations.cc
index 0371666..4142418 100644
--- a/tester/covoar/Explanations.cc
+++ b/tester/covoar/Explanations.cc
@@ -15,7 +15,6 @@
 #include <rld.h>
 
 #include "Explanations.h"
-#include "app_common.h"
 
 namespace Coverage {
 
diff --git a/tester/covoar/GcovFunctionData.cc b/tester/covoar/GcovFunctionData.cc
index 1767f18..ff0e515 100644
--- a/tester/covoar/GcovFunctionData.cc
+++ b/tester/covoar/GcovFunctionData.cc
@@ -9,7 +9,6 @@
 #include <cstring>
 #include <cinttypes>
 
-#include "app_common.h"
 #include "GcovFunctionData.h"
 #include "ObjdumpProcessor.h"
 #include "CoverageMapBase.h"
diff --git a/tester/covoar/Makefile b/tester/covoar/Makefile
index 694e4e1..bc493d8 100644
--- a/tester/covoar/Makefile
+++ b/tester/covoar/Makefile
@@ -2,8 +2,7 @@ INSTALL_DIR=../bin
 CXXFLAGS=-g -Wall -O3
 PROGRAMS=covoar qemu-dump-trace trace-converter configfile-test
 
-COMMON_OBJS= app_common.o \
-  ConfigFile.o \
+COMMON_OBJS= ConfigFile.o \
   CoverageFactory.o \
   CoverageMap.o \
   CoverageMapBase.o \
@@ -90,9 +89,6 @@ trace-converter: $(TRACECONVERTER_OBJS)
 configfile-test: $(CONFIGFILE_TEST_OBJS)
 	$(CXX) $(CXXFLAGS) -o $(@) $(CONFIGFILE_TEST_OBJS)
 
-#  DEPENDENCIES ON SINGLE OBJECTS
-app_common.o: app_common.h app_common.cc
-
 covoar.o: covoar.cc CoverageFactory.h CoverageMap.h DesiredSymbols.h \
   ExecutableInfo.h Explanations.h ObjdumpProcessor.h ReportsBase.h
 
diff --git a/tester/covoar/ObjdumpProcessor.cc b/tester/covoar/ObjdumpProcessor.cc
index 90620aa..4d1e306 100644
--- a/tester/covoar/ObjdumpProcessor.cc
+++ b/tester/covoar/ObjdumpProcessor.cc
@@ -14,7 +14,6 @@
 #include <algorithm>
 #include <string>
 
-#include "app_common.h"
 #include "ObjdumpProcessor.h"
 #include "CoverageMap.h"
 #include "ExecutableInfo.h"
diff --git a/tester/covoar/ReportsBase.cc b/tester/covoar/ReportsBase.cc
index f4df64c..bc05037 100644
--- a/tester/covoar/ReportsBase.cc
+++ b/tester/covoar/ReportsBase.cc
@@ -8,7 +8,6 @@
 #include <sstream>
 
 #include "ReportsBase.h"
-#include "app_common.h"
 #include "CoverageRanges.h"
 #include "DesiredSymbols.h"
 #include "Explanations.h"
diff --git a/tester/covoar/ReportsHtml.cc b/tester/covoar/ReportsHtml.cc
index 1b75eb0..2ba3463 100644
--- a/tester/covoar/ReportsHtml.cc
+++ b/tester/covoar/ReportsHtml.cc
@@ -9,7 +9,6 @@
 #include <rtems-utils.h>
 
 #include "ReportsHtml.h"
-#include "app_common.h"
 #include "CoverageRanges.h"
 #include "DesiredSymbols.h"
 #include "ObjdumpProcessor.h"
diff --git a/tester/covoar/ReportsText.cc b/tester/covoar/ReportsText.cc
index 6541be2..da42dab 100644
--- a/tester/covoar/ReportsText.cc
+++ b/tester/covoar/ReportsText.cc
@@ -4,7 +4,6 @@
 #include <iomanip>
 
 #include "ReportsText.h"
-#include "app_common.h"
 #include "CoverageRanges.h"
 #include "DesiredSymbols.h"
 #include "Explanations.h"
diff --git a/tester/covoar/SymbolTable.cc b/tester/covoar/SymbolTable.cc
index b3a5e58..cfbd7a3 100644
--- a/tester/covoar/SymbolTable.cc
+++ b/tester/covoar/SymbolTable.cc
@@ -13,7 +13,6 @@
 #include <rld.h>
 
 #include "SymbolTable.h"
-#include "app_common.h"
 
 namespace Coverage {
 
diff --git a/tester/covoar/TraceConverter.cc b/tester/covoar/TraceConverter.cc
index 244d0f7..0f32efb 100644
--- a/tester/covoar/TraceConverter.cc
+++ b/tester/covoar/TraceConverter.cc
@@ -21,7 +21,6 @@
 #include "TraceWriterQEMU.h"
 #include "TraceList.h"
 #include "ObjdumpProcessor.h"
-#include "app_common.h"
 #include "TargetFactory.h"
 
 #if defined(_WIN32) || defined(__CYGWIN__)
diff --git a/tester/covoar/TraceReaderLogQEMU.cc b/tester/covoar/TraceReaderLogQEMU.cc
index e65628e..57bcaa1 100644
--- a/tester/covoar/TraceReaderLogQEMU.cc
+++ b/tester/covoar/TraceReaderLogQEMU.cc
@@ -41,7 +41,6 @@
 #include <string.h>
 
 #include "qemu-log.h"
-#include "app_common.h"
 #include "TraceReaderBase.h"
 #include "TraceReaderLogQEMU.h"
 #include "TraceList.h"
diff --git a/tester/covoar/TraceWriterQEMU.cc b/tester/covoar/TraceWriterQEMU.cc
index 18a6c68..c417745 100644
--- a/tester/covoar/TraceWriterQEMU.cc
+++ b/tester/covoar/TraceWriterQEMU.cc
@@ -42,7 +42,6 @@
 
 #include <rld-process.h>
 
-#include "app_common.h"
 #include "TraceWriterQEMU.h"
 #include "ExecutableInfo.h"
 #include "CoverageMap.h"
diff --git a/tester/covoar/app_common.cc b/tester/covoar/app_common.cc
deleted file mode 100644
index 9fece7b..0000000
--- a/tester/covoar/app_common.cc
+++ /dev/null
@@ -1,49 +0,0 @@
-/*
- * RTEMS Tools Project (http://www.rtems.org/)
- * Copyright 2014 OAR Corporation
- * All rights reserved.
- *
- * This file is part of the RTEMS Tools package in 'rtems-tools'.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions are met:
- *
- * 1. Redistributions of source code must retain the above copyright notice,
- * this list of conditions and the following disclaimer.
- *
- * 2. Redistributions in binary form must reproduce the above copyright notice,
- * this list of conditions and the following disclaimer in the documentation
- * and/or other materials provided with the distribution.
- *
- * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
- * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
- * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
- * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE
- * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
- * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
- * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
- * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
- * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
- * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
- * POSSIBILITY OF SUCH DAMAGE.
- */
-
-#include "covoar-config.h"
-
-#include <stdio.h>
-#include <stdlib.h>
-#include <string.h>
-#include <sys/stat.h>
-#include <sys/types.h>
-#include <unistd.h>
-
-#include "app_common.h"
-#include "DesiredSymbols.h"
-#include "Explanations.h"
-/*
- *  Global variables for the program
- */
-
-
-
-
diff --git a/tester/covoar/app_common.h b/tester/covoar/app_common.h
deleted file mode 100644
index 5da4bb3..0000000
--- a/tester/covoar/app_common.h
+++ /dev/null
@@ -1,18 +0,0 @@
-#ifndef __APP_COMMON_h
-#define __APP_COMMON_h
-
-/*
- * This file needs to be removed and these globals removed from the
- * global scope. For example SymbolsToAnalyze is never destructed.
- */
-
-#include <list>
-
-#include "DesiredSymbols.h"
-#include "Explanations.h"
-#include "TargetBase.h"
-
-
-
-
-#endif
diff --git a/tester/covoar/coverage_converter.cc b/tester/covoar/coverage_converter.cc
index e972d83..a031395 100644
--- a/tester/covoar/coverage_converter.cc
+++ b/tester/covoar/coverage_converter.cc
@@ -11,7 +11,6 @@
 #include <sys/stat.h>
 #include <unistd.h>
 
-#include "app_common.h"
 #include "CoverageFactory.h"
 #include "CoverageMap.h"
 
diff --git a/tester/covoar/covmerge.cc b/tester/covoar/covmerge.cc
index 5ef2fe0..d09da6c 100644
--- a/tester/covoar/covmerge.cc
+++ b/tester/covoar/covmerge.cc
@@ -11,7 +11,6 @@
 #include <unistd.h>
 #include <errno.h>
 
-#include "app_common.h"
 #include "CoverageFactory.h"
 #include "CoverageMap.h"
 #include "CoverageRanges.h"
diff --git a/tester/covoar/covoar.cc b/tester/covoar/covoar.cc
index 25724b5..f9d4814 100644
--- a/tester/covoar/covoar.cc
+++ b/tester/covoar/covoar.cc
@@ -19,7 +19,6 @@
 #include <rld.h>
 #include <rld-process.h>
 
-#include "app_common.h"
 #include "CoverageFactory.h"
 #include "CoverageMap.h"
 #include "DesiredSymbols.h"
diff --git a/tester/covoar/wscript b/tester/covoar/wscript
index a928b1b..8dab4d8 100644
--- a/tester/covoar/wscript
+++ b/tester/covoar/wscript
@@ -82,8 +82,7 @@ def build(bld):
     modules = ['rld', 'dwarf', 'elf', 'iberty']
 
     bld.stlib(target = 'ccovoar',
-              source = ['app_common.cc',
-                        'AddressToLineMapper.cc',
+              source = ['AddressToLineMapper.cc',
                         'CoverageFactory.cc',
                         'CoverageMap.cc',
                         'CoverageMapBase.cc',
-- 
1.8.3.1



More information about the devel mailing list