Skip to content
This repository has been archived by the owner on Jul 15, 2023. It is now read-only.

Add contracts for System.OperatingSystem #306

Merged
merged 3 commits into from Jan 26, 2016

Conversation

yaakov-h
Copy link
Contributor

Based on MSDN and Microsoft Reference Source.

Resolves #304.

I'm not sure if I'm doing [Pure] on the getters correctly, and if VersionString should have one or not - VersionString does change an internal field, but it's only used to cache the result. There's no substantial change to the OperatingSystem object.

get
{
Contract.Ensures(!string.IsNullOrWhitespace(Contract.Result<string>()));
return default(Version);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this meant to be return default(string).

SergeyTeplyakov added a commit that referenced this pull request Jan 26, 2016
Add contracts for System.OperatingSystem
@SergeyTeplyakov SergeyTeplyakov merged commit 2285d98 into microsoft:master Jan 26, 2016
@yaakov-h yaakov-h deleted the pr-OperatingSystem branch April 24, 2018 13:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants