Remove unnecessary unsafe code flag.

[AArnott]

The Immutable PCL targets platforms that include those that don't support unsafe code. Opening the solution in VS2015 results in a warning in the error list about this. But features that required unsafe code were recently removed so we don't need this flag any more.

[AArnott]

If you notice the whitespace changes in the project file, they're natural from just saving the project in VS. So this project file was hand-edited and should (IMO) just be reverted to what VS saves by default so that we don't have to fight these whitespace changes with every checkin.

[mellinoe]

This is semi-related, but I get another warning when opening this in VS relating to one of the resx designer files. Given that you're fixing up the csproj, it might make sense to fix this as well since the project would open clean in VS then.

<Compile Include="Validation\ValidationStrings.Designer.cs">
  <DependentUpon>Validation\ValidationStrings.resx</DependentUpon>

The second line should just be

    <DependentUpon>ValidationStrings.resx</DependentUpon>.

[ellismg]

@mellinoe FWIW, I am pretty sure all that ValidationStrings stuff is dead code. I am preparing a PR that's just going to remove the whole thing.

[mellinoe]

Ah, well if that's the case then we might as well just remove it. 👍

[karelz]

@Anipik why did you change the milestone? Was it possibly mishap of the labeling tooling?

[Anipik]

yes it was mishap of the labeling tool.

[karelz]

Thanks for confirmation!