[WIP] Moving towards .NET Standard/.NET Core compatibility - #2134
alexvaluyskiy wants to merge 1 commit into
Conversation
| foreach (var typeName in typeNames) | ||
| { | ||
| var rebalanceEnabled = string.Equals(typeName, "rebalancing", StringComparison.InvariantCultureIgnoreCase); | ||
| var rebalanceEnabled = string.Equals(typeName, "rebalancing", StringComparison.OrdinalIgnoreCase); |
There was a problem hiding this comment.
MS recommends to replace InvariantCultureIgnoreCase to OrdinalIgnoreCase in many cases
|
@akkadotnet/developers So, the main question. What should we do with BinarySerialization in exceptions, and with SerializableAttribute? We could just remove it, as many projects did. Or we can add compilation directives, as I did here Akka.Net doesn't use BinarySerialization. So, why we are using these attributes? |
|
paging @JeffCyr - questions about unsafe threading on .NET Core :p |
8c465ae to
0e95adf
Compare
| { | ||
| if (CurrentInterpreter.Value[0] == null) | ||
| throw new ApplicationException("Something went terribly wrong!"); | ||
| throw new InvalidOperationException("Something went terribly wrong!"); |
There was a problem hiding this comment.
NetCore doesn't have ApplicationException, changed to InvalidOperationException
| throw new ArgumentOutOfRangeException("numThreads", string.Format("numThreads must be at least 1. Was {0}", numThreads)); | ||
| } | ||
| #else | ||
| public DedicatedThreadPoolSettings(int numThreads, string name = null, TimeSpan? deadlockTimeout = null) |
There was a problem hiding this comment.
Added a different constructor for NetCore version, without ApartmentState
|
Say it ain't so! |
|
We should create a feature branch here instead of this PR |
!Important! This PR is not about introducing NET CLI or xproj/project.json in the source code. It makes the source code easier to introduce .NET Platform compatibility in the future.
I used some ideas from this thread #992
Right now the sources match NetStandard 1.5, but it possible to achieve 1.3 in the future