- 15 8月, 2019 2 次提交
-
-
由 Fredric Silberberg 提交于
-
由 Fredric Silberberg 提交于
-
- 14 8月, 2019 1 次提交
-
-
由 Fredric Silberberg 提交于
-
- 13 8月, 2019 3 次提交
-
-
由 David Barbet 提交于
-
由 Jared Parsons 提交于
The desktop ilasm contains features not present in coreclr; specifically the ability to generate PDB files for net modules. Need to continue using desktop ilasm as long as our tests depend on this.
-
由 Jared Parsons 提交于
Co-Authored-By: NNick Guerrera <nicholg@microsoft.com>
-
- 12 8月, 2019 1 次提交
-
-
由 Jared Parsons 提交于
Including more projects that have a dependency on ilasm.
-
- 10 8月, 2019 9 次提交
-
-
由 Jared Parsons 提交于
NuPkg files are unpacked using lower case path names on Linux. This is why the build wasn't copying the files.
-
-
由 Jason Malinowski 提交于
It passes now; I'm guessing something was doing bad analysis at one point and saying something was nullable when it wasn't. Fixes https://github.com/dotnet/roslyn/issues/30035
-
由 Jared Parsons 提交于
-
由 Ivan Basov 提交于
-
由 Jason Malinowski 提交于
Sometimes other features need to light up if there's a C# or VB project present. A good example is some of the test tooling: they don't want to load their components with Roslyn unless somebody is also using the test tooling, but you don't want to accidentally load Roslyn if somebody is using the test tooling only for C++. Fixes https://devdiv.visualstudio.com/DevDiv/_queries/edit/844761/
-
由 Jared Parsons 提交于
-
由 Jared Parsons 提交于
The latest version of ilasm produced by coreclr is now a fully independent executable; it no longer depends on having a full runtime laid down next to it. This means we can vastly simplify how it is deployde in our infrastructure. The package can now be included and have ilasm manually copied out. This is similar to the approach that we take for our diasymreader native dependencies. closes #37582 related #25930, #25144
-
由 Petr Houska 提交于
-
- 09 8月, 2019 19 次提交
-
-
由 Petr Houska 提交于
-
由 Petr Houska 提交于
-
由 Petr Houska 提交于
-
由 HeeJae Chang 提交于
we can lose OOP connection due to many reason such as user explicitly killing OOP process. we have made that not to kill VS but this one place it looks like it can still crash VS. this could be due to StreamJsonRpc now throwing ConnectionLostException rather than general RemoteInvocationException it used to throw for everything. when we lost exception we do the normal pattern. showing info bar asking users to restart VS and send NFW but move on.
-
由 Ivan Basov 提交于
-
由 Jason Malinowski 提交于
When we're inferring around error types, we won't add ?s for now because they're more likely to be classes than structs, and we don't know yet exactly where the resulting syntax will get put. https://github.com/dotnet/roslyn/issues/37852 will track a fancier fix to this.
-
由 Jason Malinowski 提交于
If we have inferred the thing to the left of a ?? is a reference type, we can annotate it; we didn't update that yet in the type inferrer though until we had the ability to properly remove the ? if the user wasn't in the right context. The parent commit of this commit fixes that so we can fix this now.
-
由 Jason Malinowski 提交于
When we're generating code, we always will annotate any reference types that are known to be nullable. But we might be sticking that resulting syntax into a different file where the context disables annotations; now we will remove those ? tokens to keep code building. It's possible we could do fancier things too where we instead realize the user wants to surround the code with #nullable enable, but for now we'll take this simple approach. Fixes https://github.com/dotnet/roslyn/issues/37178
-
由 Jason Malinowski 提交于
At this point, the flighting controls we have aren't really useful anymore: we want this on for everybody unless the opt out, and that opt-out would be specific to certain users that are running into issues with the new system to keep them unblocked. I'm keeping this a per- machine setting (that doesn't roam) because it's really repo or VS version specific.
-
由 Jason Malinowski 提交于
This code as was written was a bit shaky. The Workspace constructor would ask for the workspace option service, and this would cause us (while creating the service) to call RegisterDocumentOptionsProvider. If that results in one of the options providers also asking for options, we recursively request the option service while still trying to construct it. The fix is just to move the registration to the workspace constructor itself which removes the circularity and matches the pattern used everywhere else.
-
由 Jason Malinowski 提交于
Most of the type inferrer tests test both overloads of the type inferrence service -- the one that takes a span and one that takes a position. Both (by default) were tested, but it's impossible without actually debugging a test to know which one failed. I had a particular irritating case where I made a behavior change and had to update a bunch of tests with new baselines, but since the behavior also split between both APIs it was confusing to figure out what was going on.
-
-
由 Petr Houska 提交于
-
由 Petr Houska 提交于
-
由 Petr Houska 提交于
-
由 Petr Houska 提交于
-
由 Jason Malinowski 提交于
-
由 Jason Malinowski 提交于
When generating code from the navigation bar, we should remove inaccessible attributes, since there's no value in spitting code that won't build. Fixes https://github.com/dotnet/roslyn/issues/37621 although perhaps not in the ideal way. We may also want to generally drop all nullable attributes when generating VB code, but that's being tracked by https://github.com/dotnet/roslyn/issues/30327 as there's some design questions still out.
-
由 Paul Chen 提交于
Fixes #37224 Fixes #37268 Fixes #37361
-
- 08 8月, 2019 3 次提交
-
-
由 Nikolay Balakin 提交于
-
由 Manish Vasani 提交于
This should fix the performance issues reported in VS feedback tickets [#672009](https://developercommunity.visualstudio.com/content/problem/672009/live-analysis-eats-up-memory-resulting-in-vs-crash.html) and [#672909](https://developercommunity.visualstudio.com/content/problem/672909/live-code-analysis-extremely-slow-and-vs-2019-cras.html)
-
由 Andy Gocke 提交于
There was a previous parsing change (#32999) which modified namespace parsing to allow modifiers and attributes on namespaces, to improve error recovery. This PR contained a bug because it didn't move the incremental parsing check to before parsing attributes and modifiers, which should now be included in incremental parsing to prevent changes from being dropped Fixes #37665, #37664, #37663
-
- 07 8月, 2019 2 次提交
-
-
由 Ivan Basov 提交于
-
由 Ivan Basov 提交于
Co-Authored-By: NMarius Ungureanu <teromario@yahoo.com>
-