Support around JSON-RPC (#2014)

* #include an absolute path didn't work - because paths were taken to always be relative.

* Use 'Process' to communicate with an command line tool.

* Remove slang-win-stream

* Tidy up windows ProcessUtil.

* First version of BufferedReadStream.

* Windows working IPC for steams.

* Test proxy count option.

* Split Process/ProcessUtil. Process is platform dependant. ProcessUtil are functions that are platform independent.

* First implementation of Unix Process interface.

* Unix process compiles on cygwin.

* Fix typo in unix process.

* Separate unix pipe stream error of invalid access, from pipe availability.

* Fix in standard line extraction.

* Make fd non blocking.

* Fix issues with Windows Process streams.

* Added UnixPipe.

* Some fixes around UnixPipeStream.

* Make a unix stream closed explicit.

* Hack to debug linux process/stream.

* Revert to old linux pipe handling.

* Pass executable path for unit tests.
Split out CommandLine into own source.

* Small improvements in process/command line.

* Check process behavior with crash.

* Make stderr and stdout unbuffered for crash testing.

* Only turn disable buffering in crash test.

* Disable crash test on CI.

* Fix crash on clang/linux.

* Enable crash test.
Remove _appendBuffer as can use StreamUtil functionality.

* Added inital processing for http headers.

* Small improvements to HttpHeader.

* First pass HTTPPacketConnection working on windows.

* Enable other Process communication tests.

* Update comments.

* WIP JSON RPC.

* Add terminate to Process.
Made JSONRPC a Util.

* Small tidy up around HTTPPacketConnection.

* Improve process termination options.

Co-authored-by: Yong He <yonghe@outlook.com>

1 files changed, 25 insertions, 3 deletions

diff --git a/source/compiler-core/slang-source-loc.cpp b/source/compiler-core/slang-source-loc.cpp
index 29fc0465a..3033d9626 100644
--- a/source/compiler-core/slang-source-loc.cpp
+++ b/source/compiler-core/slang-source-loc.cpp
@@ -425,14 +425,24 @@ void SourceManager::initialize(
 
     m_parent = p;
 
-    if( p )
+    _resetLoc();
+}
+
+SourceManager::~SourceManager()
+{
+    _resetSource();
+}
+
+void SourceManager::_resetLoc()
+{
+    if (m_parent)
     {
         // If we have a parent source manager, then we assume that all code at that level
         // has already been loaded, and it is safe to start our own source locations
         // right after those from the parent.
         //
         // TODO: more clever allocation in cases where that might not be reasonable
-        m_startLoc = p->m_nextLoc;
+        m_startLoc = m_parent->m_nextLoc;
     }
     else
     {
@@ -444,7 +454,7 @@ void SourceManager::initialize(
     m_nextLoc = m_startLoc;
 }
 
-SourceManager::~SourceManager()
+void SourceManager::_resetSource()
 {
     for (auto item : m_sourceViews)
     {
@@ -455,6 +465,18 @@ SourceManager::~SourceManager()
     {
         delete item;
     }
+
+    m_sourceViews.clear();
+    m_sourceFiles.clear();
+
+    m_sourceFileMap.Clear();
+}
+
+
+void SourceManager::reset()
+{
+    _resetSource();
+    _resetLoc();
 }
 
 UnownedStringSlice SourceManager::allocateStringSlice(const UnownedStringSlice& slice)