Probably a bug with CS8302

Steps to Reproduce:

Use #error version:7.1 with LangVersion set to 7.1

Expected Behavior:

No diagnostic

Actual Behavior:

error CS8302: Feature 'version' is not available in C# 7.1. Please use language version 7.1 or greater.

The error message looks very strange.

The problem lies in DirectiveParser.cs, more specifically, this part of code:

if (this.Options.LanguageVersion != LanguageVersion.Preview &&
    errorText.StartsWith(versionMarker, StringComparison.Ordinal) &&
    LanguageVersionFacts.TryParse(errorText.Substring(versionMarker.Length), out var languageVersion))
{
    ErrorCode error = this.Options.LanguageVersion.GetErrorCode();
    eod = this.AddError(eod, triviaOffset, triviaWidth, error, "version", new CSharpRequiredLanguageVersion(languageVersion));
}

I think the if condition needs to compare the parsed languageVersion with this.Options.LanguageVersion or this.Options.SpecifiedLanguageVersion (not sure what’s the difference between them, hope someone can clear this up for me).

[jcouv]

Since #error version:<version> is not meant for users, I don’t worry about the quality of the diagnostic message. I’ll go ahead and close this issue as by-design. Thanks

[RikkiGibson]

Whoops, I recently added a PR in dotnet/docs adding information about #error version. Should I revert it?

I think it’s fine.