Deukhoofd/Upsilon
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Make sure condition for if statement is a boolean

Browse Source
This commit is contained in:
Deukhoofd
2019-01-31 15:18:21 +01:00
parent 2e08669509
commit 947904f097
1 changed files with 5 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
Upsilon/Binder/Binder.cs
View File

@@ -60,12 +60,7 @@ namespace Upsilon.Binder
} }
unboundFunctionStatement.Block = unboundFunctionStatement.Block =
(BoundBlockStatement) BindBlockStatement(unboundFunctionStatement.UnboundBlock); (BoundBlockStatement) BindBlockStatement(unboundFunctionStatement.UnboundBlock);
var resultType = Scope.ReturnType;
Scope = Scope.ParentScope; Scope = Scope.ParentScope;
//var variable =
// (ScriptFunctionVariableSymbol) unboundFunctionStatement.Scope.ParentScope.Variables[
// unboundFunctionStatement];
//variable.IsBound = true;
} }
_unboundFunctions.Clear(); _unboundFunctions.Clear();
return new BoundScript((BoundBlockStatement) bound, e.Span, Scope, fileName, _script); return new BoundScript((BoundBlockStatement) bound, e.Span, Scope, fileName, _script);
@@ -465,7 +460,7 @@ namespace Upsilon.Binder
return new BoundVariableExpression(boundVariable, e.Span); return new BoundVariableExpression(boundVariable, e.Span);
} }
private BoundStatement BindExpressionStatement(ExpressionStatementSyntax s) private BoundExpressionStatement BindExpressionStatement(ExpressionStatementSyntax s)
{ {
var exp = BindExpression(s.Expression); var exp = BindExpression(s.Expression);
return new BoundExpressionStatement(exp, s.Span); return new BoundExpressionStatement(exp, s.Span);
@@ -620,6 +615,10 @@ namespace Upsilon.Binder
{ {
Scope = new BoundScope(Scope); Scope = new BoundScope(Scope);
var condition = BindExpressionStatement(e.Condition); var condition = BindExpressionStatement(e.Condition);
if (condition.Expression.Type != Type.Boolean && condition.Expression.Type != Type.Unknown)
{
_diagnostics.LogError("Condition for if statement should be a boolean.", condition.Span);
}
var block = BindBlockStatement(e.Block); var block = BindBlockStatement(e.Block);
Scope = Scope.ParentScope; Scope = Scope.ParentScope;
if (e.NextElseIfStatement != null) if (e.NextElseIfStatement != null)