Merge pull request #356 from LeStarch/fix/template-implementation-paths

Fixing implementation include paths

Author: bocchino

compiler/lib/src/main/scala/codegen/CppWriter/ComponentCppWriter/ComponentImplWriter.scala

@@ -50,7 +50,7 @@ case class ComponentImplWriter(
   private def getCppIncludes: CppDoc.Member = {
     val userHeaders = List(
       "FpConfig.hpp",
-      s"${s.getRelativePath(fileName).toString}.hpp"
+      s.getIncludePath(symbol, fileName).toString
     )
     linesMember(
       addBlankPrefix(userHeaders.sorted.map(CppWriter.headerString).map(line)),

compiler/lib/src/main/scala/codegen/CppWriter/ComponentCppWriter/TestWriter/ComponentTestImplWriter.scala

@@ -40,9 +40,9 @@ case class ComponentTestImplWriter(
 
   private def getHppIncludes: CppDoc.Member = {
     val headers = List(
-      s"${s.getRelativePath(ComputeCppFiles.FileNames.getComponentGTestBase(name))}.hpp",
-      s"${s.getRelativePath(ComputeCppFiles.FileNames.getComponentImpl(name))}.hpp"
-    )
+      ComputeCppFiles.FileNames.getComponentGTestBase(name),
+      ComputeCppFiles.FileNames.getComponentImpl(name)
+    ).map(s.getIncludePath(symbol, _))
     linesMember(
       addBlankPrefix(headers.map(CppWriter.headerString).map(line))
     )

compiler/lib/src/main/scala/codegen/CppWriter/CppWriterState.scala

@@ -128,10 +128,19 @@ case class CppWriterState(
       removeComponentQualifiers(psOpt, out1)
   }
 
-  /** Write include directives as lines */
+  /** Get an include path for a symbol and a file name base */
+  def getIncludePath(
+    sym: Symbol,
+    fileNameBase: String
+  ): String = {
+    val loc = sym.getLoc.tuLocation
+    val fullPath = loc.getNeighborPath(fileNameBase)
+    val path = removeLongestPathPrefix(fullPath)
+    s"${path.toString}.hpp"
+  }
+
+  /** Write include directives for autocoded files */
   def writeIncludeDirectives(usedSymbols: Iterable[Symbol]): List[String] = {
-    def getHeaderStr(file: File.JavaPath) =
-      CppWriter.headerString(s"${file.toString}.hpp")
     def getDirectiveForSymbol(sym: Symbol): Option[String] =
       for {
         fileName <- sym match {
@@ -160,12 +169,7 @@ case class CppWriterState(
           case _ => None
         }
       }
-      yield {
-        val loc = sym.getLoc.tuLocation
-        val fullPath = loc.getNeighborPath(fileName)
-        val path = removeLongestPathPrefix(fullPath)
-        getHeaderStr(path)
-      }
+      yield CppWriter.headerString(getIncludePath(sym, fileName))
 
     usedSymbols.map(getDirectiveForSymbol).filter(_.isDefined).map(_.get).toList
   }

compiler/tools/fpp-to-cpp/test/component/impl/ActiveAsyncProductPortsOnly.template.ref.cpp

@@ -4,8 +4,8 @@
 // \brief  cpp file for ActiveAsyncProductPortsOnly component implementation class
 // ======================================================================
 
+#include "ActiveAsyncProductPortsOnly.hpp"
 #include "FpConfig.hpp"
-#include "impl/ActiveAsyncProductPortsOnly.hpp"
 
 // ----------------------------------------------------------------------
 // Component construction and destruction

compiler/tools/fpp-to-cpp/test/component/impl/ActiveAsyncProducts.template.ref.cpp

@@ -4,8 +4,8 @@
 // \brief  cpp file for ActiveAsyncProducts component implementation class
 // ======================================================================
 
+#include "ActiveAsyncProducts.hpp"
 #include "FpConfig.hpp"
-#include "impl/ActiveAsyncProducts.hpp"
 
 // ----------------------------------------------------------------------
 // Component construction and destruction

compiler/tools/fpp-to-cpp/test/component/impl/ActiveCommands.template.ref.cpp

@@ -4,8 +4,8 @@
 // \brief  cpp file for ActiveCommands component implementation class
 // ======================================================================
 
+#include "ActiveCommands.hpp"
 #include "FpConfig.hpp"
-#include "impl/ActiveCommands.hpp"
 
 // ----------------------------------------------------------------------
 // Component construction and destruction

compiler/tools/fpp-to-cpp/test/component/impl/ActiveEvents.template.ref.cpp

@@ -4,8 +4,8 @@
 // \brief  cpp file for ActiveEvents component implementation class
 // ======================================================================
 
+#include "ActiveEvents.hpp"
 #include "FpConfig.hpp"
-#include "impl/ActiveEvents.hpp"
 
 // ----------------------------------------------------------------------
 // Component construction and destruction

compiler/tools/fpp-to-cpp/test/component/impl/ActiveGetProducts.template.ref.cpp

@@ -4,8 +4,8 @@
 // \brief  cpp file for ActiveGetProducts component implementation class
 // ======================================================================
 
+#include "ActiveGetProducts.hpp"
 #include "FpConfig.hpp"
-#include "impl/ActiveGetProducts.hpp"
 
 // ----------------------------------------------------------------------
 // Component construction and destruction

compiler/tools/fpp-to-cpp/test/component/impl/ActiveGuardedProducts.template.ref.cpp

@@ -4,8 +4,8 @@
 // \brief  cpp file for ActiveGuardedProducts component implementation class
 // ======================================================================
 
+#include "ActiveGuardedProducts.hpp"
 #include "FpConfig.hpp"
-#include "impl/ActiveGuardedProducts.hpp"
 
 // ----------------------------------------------------------------------
 // Component construction and destruction

compiler/tools/fpp-to-cpp/test/component/impl/ActiveParams.template.ref.cpp

@@ -4,8 +4,8 @@
 // \brief  cpp file for ActiveParams component implementation class
 // ======================================================================
 
+#include "ActiveParams.hpp"
 #include "FpConfig.hpp"
-#include "impl/ActiveParams.hpp"
 
 // ----------------------------------------------------------------------
 // Component construction and destruction