sms/ssc
1
0
Fork 0
Code Issues Pull requests Releases 2 Activity

Refactored break/continue compilation.

Browse source 
I didn't like the null overload usage. I might remove the overload
completely later.
This commit is contained in:
arookas 2016-01-31 21:25:50 -05:00
parent eb3e679fc7
commit b2330dfa6c
1 changed files with 16 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
ssc/ast/nodes.flow.cs
View file

@ -155,7 +155,14 @@ namespace arookas {
public override void Compile(sunCompiler compiler) {
var point = compiler.Binary.WriteJMP();
if (!compiler.Context.Loops.AddBreak(point, IsNamed ? NameLabel.Value : null)) {
var success = true;
if (IsNamed) {
success = compiler.Context.Loops.AddBreak(point, NameLabel.Value);
}
else {
success = compiler.Context.Loops.AddBreak(point);
}
if (!success) {
throw new sunBreakException(this);
}
}
@ -170,7 +177,14 @@ namespace arookas {
public override void Compile(sunCompiler compiler) {
var point = compiler.Binary.WriteJMP();
if (!compiler.Context.Loops.AddContinue(point, IsNamed ? NameLabel.Value : null)) {
var success = true;
if (IsNamed) {
success = compiler.Context.Loops.AddContinue(point, NameLabel.Value);
}
else {
success = compiler.Context.Loops.AddContinue(point);
}
if (!success) {
throw new sunContinueException(this);
}
}