Skip to content

Commit

Permalink
bunch of more tests fixed... 6 failing
Browse files Browse the repository at this point in the history
  • Loading branch information
retailcoder committed Jan 22, 2018
1 parent 47ffe0b commit dc927b7
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions Rubberduck.Parsing/Symbols/IdentifierReferenceResolver.cs
Expand Up @@ -194,9 +194,9 @@ private void ResolveLabel(ParserRuleContext context, string label)
{
var module = boundExpression.ReferencedDeclaration.AsTypeDeclaration;
var members = _declarationFinder.Members(module);
defaultMember = (IParameterizedDeclaration)members.SingleOrDefault(m => m is IParameterizedDeclaration && m.Attributes.Any(a => a.Name == $"{m.IdentifierName}.VB_UserMemId" && a.Values.SingleOrDefault() == "0"));
defaultMember = (IParameterizedDeclaration)members.SingleOrDefault(m => m is IParameterizedDeclaration && m.Attributes.HasDefaultMemberAttribute());
}
_boundExpressionVisitor.AddIdentifierReferences(boundExpression, _qualifiedModuleName, _currentScope, _currentParent, isAssignmentTarget && (defaultMember != null && (!defaultMember.Parameters.Any() || defaultMember.Parameters.All(p => p.IsOptional)) || isSetAssignment), hasExplicitLetStatement);
_boundExpressionVisitor.AddIdentifierReferences(boundExpression, _qualifiedModuleName, _currentScope, _currentParent, isAssignmentTarget && (defaultMember == null || (!defaultMember.Parameters.Any() || defaultMember.Parameters.All(p => p.IsOptional)) || isSetAssignment), hasExplicitLetStatement);
}

private void ResolveType(ParserRuleContext expression)
Expand Down
Expand Up @@ -14,8 +14,8 @@ public class UnassignedVariableUsageInspectionTests
[Category("Inspections")]
public void UnassignedVariableUsage_ReturnsResult()
{
const string inputCode =
@"Sub Foo()
const string inputCode = @"
Sub Foo()
Dim b As Boolean
Dim bb As Boolean
bb = b
Expand Down

0 comments on commit dc927b7

Please sign in to comment.