Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
lwm1986
roslyn
提交
2e2cbb6c
R
roslyn
项目概览
lwm1986
/
roslyn
与 Fork 源项目一致
从无法访问的项目Fork
通知
1
Star
0
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
DevOps
流水线
流水线任务
计划
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
R
roslyn
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
DevOps
DevOps
流水线
流水线任务
计划
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
流水线任务
提交
Issue看板
体验新版 GitCode,发现更多精彩内容 >>
提交
2e2cbb6c
编写于
1月 11, 2018
作者:
M
Martin Strecker
提交者:
Sam Harwell
2月 14, 2018
浏览文件
操作
浏览文件
下载
电子邮件补丁
差异文件
Add HasUnconstraintGenericParameter check to suppress reporting of diagnostic.
上级
c05e6fb4
变更
2
隐藏空白更改
内联
并排
Showing
2 changed file
with
73 addition
and
2 deletion
+73
-2
src/EditorFeatures/CSharpTest/UseIsNullCheck/UseIsNullCheckTests.cs
...Features/CSharpTest/UseIsNullCheck/UseIsNullCheckTests.cs
+50
-0
src/Features/Core/Portable/UseIsNullCheck/AbstractUseIsNullDiagnosticAnalyzer.cs
...ble/UseIsNullCheck/AbstractUseIsNullDiagnosticAnalyzer.cs
+23
-2
未找到文件。
src/EditorFeatures/CSharpTest/UseIsNullCheck/UseIsNullCheckTests.cs
浏览文件 @
2e2cbb6c
...
...
@@ -217,5 +217,55 @@ void M(string s1, string s2)
}
}"
);
}
[
WorkItem
(
23581
,
"https://github.com/dotnet/roslyn/issues/23581"
)]
[
Fact
,
Trait
(
Traits
.
Feature
,
Traits
.
Features
.
CodeActionsUseIsNullCheck
)]
public
async
Task
TestMissingIfValueParameterTypeIsUnconstraintGeneric
()
{
await
TestMissingAsync
(
@"
class C
{
public static void NotNull<T>(T value, string parameterName)
{
if ([||]ReferenceEquals(value, null))
{
throw new System.ArgumentNullException(parameterName);
}
}
}
"
);
}
[
WorkItem
(
23581
,
"https://github.com/dotnet/roslyn/issues/23581"
)]
[
Fact
,
Trait
(
Traits
.
Feature
,
Traits
.
Features
.
CodeActionsUseIsNullCheck
)]
public
async
Task
TestValueParameterTypeIsConstraintGeneric
()
{
await
TestInRegularAndScriptAsync
(
@"
class C
{
public static void NotNull<T>(T value, string parameterName) where T:class
{
if ([||]ReferenceEquals(value, null))
{
throw new System.ArgumentNullException(parameterName);
}
}
}
"
,
@"
class C
{
public static void NotNull<T>(T value, string parameterName) where T:class
{
if (value is null)
{
throw new System.ArgumentNullException(parameterName);
}
}
}
"
);
}
}
}
src/Features/Core/Portable/UseIsNullCheck/AbstractUseIsNullDiagnosticAnalyzer.cs
浏览文件 @
2e2cbb6c
...
...
@@ -2,6 +2,7 @@
using
System.Collections.Immutable
;
using
System.Linq
;
using
System.Threading
;
using
Microsoft.CodeAnalysis.CodeStyle
;
using
Microsoft.CodeAnalysis.Diagnostics
;
using
Microsoft.CodeAnalysis.LanguageServices
;
...
...
@@ -49,7 +50,7 @@ protected override void InitializeWorker(AnalysisContext context)
private
void
AnalyzeSyntax
(
SyntaxNodeAnalysisContext
context
,
IMethodSymbol
referenceEqualsMethod
)
{
var
cancellationToken
=
context
.
CancellationToken
;
var
semanticModel
=
context
.
SemanticModel
;
var
syntaxTree
=
semanticModel
.
SyntaxTree
;
if
(!
IsLanguageVersionSupported
(
syntaxTree
.
Options
))
...
...
@@ -108,6 +109,10 @@ private void AnalyzeSyntax(SyntaxNodeAnalysisContext context, IMethodSymbol refe
return
;
}
if
(
HasUnconstraintGenericParameter
(
syntaxFacts
,
semanticModel
,
arguments
[
0
],
arguments
[
1
],
cancellationToken
))
{
return
;
}
var
additionalLocations
=
ImmutableArray
.
Create
(
invocation
.
GetLocation
());
var
properties
=
ImmutableDictionary
<
string
,
string
>.
Empty
;
...
...
@@ -125,7 +130,23 @@ private void AnalyzeSyntax(SyntaxNodeAnalysisContext context, IMethodSymbol refe
additionalLocations
,
properties
));
}
private
bool
MatchesPattern
(
ISyntaxFactsService
syntaxFacts
,
SyntaxNode
node1
,
SyntaxNode
node2
)
private
static
bool
HasUnconstraintGenericParameter
(
ISyntaxFactsService
syntaxFacts
,
SemanticModel
semanticModel
,
SyntaxNode
node1
,
SyntaxNode
node2
,
CancellationToken
cancellationToken
)
{
var
valueNode
=
syntaxFacts
.
IsNullLiteralExpression
(
node1
)
?
node2
:
node1
;
var
argumentExpression
=
syntaxFacts
.
GetExpressionOfArgument
(
valueNode
);
if
(
argumentExpression
!=
null
)
{
var
parameterType
=
semanticModel
.
GetTypeInfo
(
argumentExpression
,
cancellationToken
).
Type
;
if
(
parameterType
is
ITypeParameterSymbol
typeParameter
)
{
return
!
typeParameter
.
HasReferenceTypeConstraint
;
}
}
return
false
;
}
private
static
bool
MatchesPattern
(
ISyntaxFactsService
syntaxFacts
,
SyntaxNode
node1
,
SyntaxNode
node2
)
=>
syntaxFacts
.
IsNullLiteralExpression
(
syntaxFacts
.
GetExpressionOfArgument
(
node1
))
&&
!
syntaxFacts
.
IsNullLiteralExpression
(
syntaxFacts
.
GetExpressionOfArgument
(
node2
));
}
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录