Skip to content

Commit 3b4900b

Browse files
author
Kapil Borle
committed
Clean up inline comments
1 parent b807b9a commit 3b4900b

File tree

1 file changed

+1
-11
lines changed

1 file changed

+1
-11
lines changed

Rules/UseShouldProcessCorrectly.cs

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -170,7 +170,7 @@ private DiagnosticRecord GetViolation(Vertex v)
170170
{
171171
if (callsShouldProcess)
172172
{
173-
// check if upstream function declares SupportShouldProcess\
173+
// check if upstream function declares SupportShouldProcess
174174
// if so, this might just be a helper function
175175
// do not flag this case
176176
if (UpstreamDeclaresShouldProcess(v))
@@ -515,16 +515,6 @@ public override AstVisitAction VisitCommand(CommandAst ast)
515515
return AstVisitAction.Continue;
516516
}
517517

518-
// if command is part of a binary module
519-
// for now just check if (Get-Command <CommandName>).DLL end with dll extension
520-
// if so, check if it declares SupportsShouldProcess
521-
// if so, then assume it also calls ShouldProcess
522-
// because we do not have a way to analyze its definition
523-
// to actually verify it is indeed calling ShouddProcess
524-
525-
// if (IsPartOfBinaryModule(cmdName, out cmdInfo))
526-
// if (HasSupportShouldProcessAttribute(cmdInfo))
527-
// AddEdge(cmdName, shouldProcessVertex)
528518
var vertex = new Vertex (cmdName, ast);
529519
AddVertex(vertex);
530520
if (IsWithinFunctionDefinition())

0 commit comments

Comments
 (0)