Merge pull request #106 from rainers/master

Changes for 0.48.1

Author: rainers

CHANGES

@@ -986,7 +986,7 @@ Version history
   * miscellaneous
     - Issue 18622: removed outdated information regarding link definition when generated by Visual D DLL project.
 
-unreleased Version 0.48.0
+2018-12-02 Version 0.48.0
 
   * installation
     - installer and binaries now digitally signed by the "D Language Foundation"
@@ -1055,3 +1055,19 @@ unreleased Version 0.48.0
     - added option to not indent case statements
   * fixed "Compile and Run" on selection
   * fix help via F1 for dmd 2.072+
+
+2018-12-22 Version 0.48.1
+  * dparser:
+    - fix Issue 19411: increase stack size for DParserCOMServer
+    - goto definition on alias: jump to base definition if caret on alias
+  * build system
+    - fix Issue 19445: Linking to lib under "c:\Program Files (x86)" does not work in Visual D
+  * dustmite
+    - fix copying project for dustmite with relative paths
+    - move command line options to new setup page, add dustmite command line options
+  * cv2pdb
+    - fix crash if mspdb*.dll cannot be loaded from path
+    - emit "elemtype[keytype]" instead of "dAssocArray"
+    - fix missing line numbers for new versions of mspdb (e.g. 14.16.27012/VS 15.9)
+  * mago: variables captured from stack of outer function not displayed in locals
+

Makefile

@@ -54,6 +54,7 @@ vdserver:
 
 dparser:
 	cd vdc\abothe && $(MSBUILD15) vdserver.sln /p:Configuration=Release;Platform="Any CPU" /p:TargetFrameworkVersion=4.5 /p:DefineConstants=NET40 /t:Rebuild
+	editbin /STACK:0x800000 bin\Release\DParserCOMServer\DParserCOMServer.exe
 
 dparser_test:
 	set PLATFORM="Any CPU" && dotnet test vdc\abothe\VDServer.sln -c Release

VERSION

@@ -1,5 +1,5 @@
 #define VERSION_MAJOR      0
 #define VERSION_MINOR      48
-#define VERSION_REVISION   0
+#define VERSION_REVISION   1
 #define VERSION_BETA       
 #define VERSION_BUILD      0

doc/StartPage.dd

@@ -54,6 +54,13 @@ $(H2 News)
 $(P $(LINK2 VersionHistory.html, Full version history and complete details...)
 )
 
+2018-12-22 Version 0.48.1
+$(UL
+  $(LI goto definition on alias: jump to base definition if caret on alias)
+  $(LI cv2pdb: fix missing line numbers for new versions of mspdb*.dll)
+  $(LI mago: variables captured from stack of outer function not displayed in locals)
+)
+
 2018-12-02 Version 0.48.0
 $(UL
   $(LI installer and binaries now digitally signed by the "D Language Foundation")

doc/VersionHistory.dd

@@ -1,5 +1,34 @@
 Ddoc
 
+$(H2 2018-12-22 Version 0.48.1)
+ $(UL
+  $(LI dparser:
+   $(UL
+    $(LI fix Issue 19411: increase stack size for DParserCOMServer)
+    $(LI goto definition on alias: jump to base definition if caret on alias)
+  ))
+  $(LI build system
+   $(UL
+    $(LI fix Issue 19445: Linking to lib under "c:\Program Files (x86)" does not work in Visual D)
+  ))
+  $(LI dustmite
+   $(UL
+    $(LI fix copying project for dustmite with relative paths)
+    $(LI move command line options to new setup page, add dustmite command line options)
+  ))
+  $(LI cv2pdb
+   $(UL
+    $(LI fix crash if mspdb*.dll cannot be loaded from path)
+    $(LI emit "elemtype[keytype]" instead of "dAssocArray")
+    $(LI fix missing line numbers for new versions of mspdb*.dll (e.g. 14.16.27012/VS 15.9))
+  ))
+  $(LI mago
+   $(UL
+     $(LI variables captured from stack of outer function not displayed in locals)
+  ))
+ )
+
+
 $(H2 2018-12-02 Version 0.48.0)
  $(UL
   $(LI installation

vdc/abothe/comserver/CodeSemantics/SymbolDefinitionGenerator.cs

@@ -27,6 +27,9 @@ protected override Tuple<CodeLocation, CodeLocation, string> Process(EditorData
 			var definitionSourceFilename = new StringBuilder();
 			if (rr != null)
 			{
+				var at = rr as AliasedType;
+				if (at != null && at.Definition.Location == sr.Location)
+					rr = at.Base;
 				DNode n = null;
 				foreach (var t in AmbiguousType.TryDissolve(rr))
 				{

visuald/chiernode.d

@@ -60,7 +60,8 @@ class CHierNode : DisposingDispatchObject
 {
 	static void shared_static_this()
 	{
-		gVsItemMap_sync = new Object;
+		if (!gVsItemMap_sync)
+			gVsItemMap_sync = new Object;
 	}
 
 	this()

visuald/config.d

@@ -963,6 +963,7 @@ class ProjectOptions
 			string s;
 			foreach(i, file; lnkfiles)
 			{
+				file = unquoteArgument(file);
 				if(toLower(extension(file)) != ext)
 					continue;
 				if(s.length > 0)

visuald/dpackage.d

@@ -407,6 +407,8 @@ class Package : DisposingComObject,
 			tpp = newCom!LdcDirPropertyPage(mOptions);
 		else if(*rguidPage == g_DubPropertyPage)
 			tpp = newCom!DubPropertyPage(mOptions);
+		else if(*rguidPage == g_CmdLinePropertyPage)
+			tpp = newCom!CmdLinePropertyPage(mOptions);
 		else if(*rguidPage == g_ToolsProperty2Page)
 			tpp = newCom!ToolsProperty2Page(mOptions);
 		else if(*rguidPage == g_ColorizerPropertyPage)
@@ -1353,6 +1355,7 @@ class GlobalOptions
 	string compileAndRunOpts;
 	string compileAndDbgOpts;
 	int compileAndDbgEngine;
+	string dustmiteOpts;
 
 	string[] coverageBuildDirs;
 	string[] coverageExecutionDirs;
@@ -1837,6 +1840,7 @@ class GlobalOptions
 			compileAndRunOpts = getStringOpt("compileAndRunOpts", "-unittest");
 			compileAndDbgOpts = getStringOpt("compileAndDbgOpts", "-g");
 			compileAndDbgEngine = getIntOpt("compileAndDbgEngine", 0);
+			dustmiteOpts      = getStringOpt("dustmiteOpts", "--strip-comments --split *.bat:lines");
 
 			dubPath           = getStringOpt("dubPath", "dub");
 			dubOptions        = getStringOpt("dubOptions", "");
@@ -1961,6 +1965,7 @@ class GlobalOptions
 			keyToolOpts.Set("compileAndRunOpts",   toUTF16(compileAndRunOpts));
 			keyToolOpts.Set("compileAndDbgOpts",   toUTF16(compileAndDbgOpts));
 			keyToolOpts.Set("compileAndDbgEngine", compileAndDbgEngine);
+			keyToolOpts.Set("dustmiteOpts",        toUTF16(dustmiteOpts));
 
 			keyToolOpts.Set("coverageExecutionDirs", toUTF16(join(coverageExecutionDirs, ";")));
 			keyToolOpts.Set("coverageBuildDirs",   toUTF16(join(coverageBuildDirs, ";")));

visuald/dproject.d

@@ -2798,7 +2798,7 @@ HRESULT DustMiteProject()
 	if (msgRet != IDYES)
 		return S_FALSE;
 
-	string workdir = cfg.GetProjectDir();
+	string workdir = normalizeDir(cfg.GetProjectDir());
 
 	auto pane = getVisualDOutputPane();
 	scope(exit) release(pane);
@@ -2822,7 +2822,9 @@ HRESULT DustMiteProject()
 		}
 		pane.OutputString(_toUTF16z("created clean copy of the project in " ~ dustmitepath ~ "\n"));
 
-		nworkdir = npath; // TODO
+		auto reldir = makeRelative(workdir, commonpath);
+		npath = makeDirnameCanonical(npath, null);
+		nworkdir = makeDirnameCanonical(reldir, npath);
 		string nintdir = makeFilenameAbsolute(cfg.GetIntermediateDir(), nworkdir);
 		string noutdir = makeFilenameAbsolute(cfg.GetOutDir(), nworkdir);
 		mkdirRecurse(nworkdir);
@@ -2833,7 +2835,7 @@ HRESULT DustMiteProject()
 		std.file.write(normalizeDir(noutdir) ~ "empty.txt", "");
 
 		if (nworkdir != npath)
-			cmdline ~= "cd " ~ quoteFilename(makeRelative(nworkdir, npath));
+			cmdline ~= "cd " ~ quoteFilename(makeRelative(nworkdir, npath)) ~ "\n";
 		cmdline ~= cfg.getCommandLine(true, true, false);
 		cmdfile = npath ~ "build.dustmite.bat";
 		std.file.write(cmdfile, cmdline);
@@ -2844,7 +2846,7 @@ HRESULT DustMiteProject()
 		string intdir = makeFilenameAbsolute(cfg.GetIntermediateDir(), workdir);
 		mkdirRecurse(intdir);
 		dustfile = intdir ~ "\\dustmite.cmd";
-		string opts = "--strip-comments --split *.bat:lines";
+		string opts = Package.GetGlobalOptions().dustmiteOpts;
 		cmd = Package.GetGlobalOptions().findDmdBinDir() ~ "dustmite " ~ opts ~ " " ~ quoteFilename(npath[0..$-1]) ~ " \"" ~ dustcmd ~ "\"";
 		std.file.write(dustfile, cmd ~ "\npause\n");
 		std.process.spawnShell(quoteFilename(dustfile), null, std.process.Config.none, nworkdir);
@@ -2876,3 +2878,41 @@ class DustMiteThread : CBuilderThread
 
 	string mBuildDir;
 }
+
+unittest
+{
+	CHierNode.shared_static_this();
+	Package pkg = newCom!Package.addref();
+	scope(exit) pkg.release();
+	string tmpdir = normalizeDir(tempDir());
+	string projdir = tmpdir ~ "test/dustmite/project/";
+	string projfile = projdir ~ "project.vdproj";
+	string platform = "Win32", config = "Debug";
+	auto cfg = newCom!VCConfig(projfile, "test", platform, config);
+
+	Project prj = newCom!Project(Package.GetProjectFactory(), "test", projfile, platform, config);
+
+	mkdirRecurse(projdir);
+	string dfile1 = projdir ~ "dfile1.d";
+	std.file.write(dfile1, "// dfile1\n");
+	auto pFile1 = newCom!CFileNode("dfile1.d");
+	prj.GetRootNode().AddTail(pFile1);
+
+	mkdirRecurse(tmpdir ~ "test/dustmite/other");
+	string dfile2 = tmpdir ~ "test/dustmite/other/dfile2.d";
+	std.file.write(dfile2, "// dfile2\n");
+	auto pFile2 = newCom!CFileNode("../other/dfile2.d");
+	prj.GetRootNode().AddTail(pFile2);
+
+	string commonpath = commonProjectFolder(prj);
+	string dustmitepath = buildPath(dirName(commonpath), baseName(commonpath) ~ ".dustmite");
+	assert(dustmitepath == tmpdir ~ r"test\dustmite.dustmite");
+	if (std.file.exists(dustmitepath))
+		rmdirRecurse(dustmitepath);
+
+	string npath = copyProjectFolder(prj, dustmitepath);
+	assert(npath == normalizeDir(dustmitepath));
+
+	assert(std.file.exists(dustmitepath ~ r"\project\dfile1.d"));
+	assert(std.file.exists(dustmitepath ~ r"\other\dfile2.d"));
+}