diff options
author | Rico Tzschichholz <ricotz@ubuntu.com> | 2018-01-07 11:25:03 +0100 |
---|---|---|
committer | Rico Tzschichholz <ricotz@ubuntu.com> | 2018-01-16 18:26:50 +0100 |
commit | 55a89521f43b4d01e6a98f7ceadc632b04fe2dd5 (patch) | |
tree | 495266087b95b0d756bbbd1251892e05fef1f1e7 | |
parent | a8327c61be67fed0c299c7ee4ab05920f8b77a7b (diff) | |
download | vala-55a89521f43b4d01e6a98f7ceadc632b04fe2dd5.tar.gz |
vala: Replace if-else-tree with switch
-rw-r--r-- | vala/valaassignment.vala | 31 |
1 files changed, 11 insertions, 20 deletions
diff --git a/vala/valaassignment.vala b/vala/valaassignment.vala index fc27acf63..b1bc4dc53 100644 --- a/vala/valaassignment.vala +++ b/vala/valaassignment.vala @@ -237,26 +237,17 @@ public class Vala.Assignment : Expression { right.target_type = right.target_type.copy (); right.target_type.value_owned = false; - if (operator == AssignmentOperator.BITWISE_OR) { - bin.operator = BinaryOperator.BITWISE_OR; - } else if (operator == AssignmentOperator.BITWISE_AND) { - bin.operator = BinaryOperator.BITWISE_AND; - } else if (operator == AssignmentOperator.BITWISE_XOR) { - bin.operator = BinaryOperator.BITWISE_XOR; - } else if (operator == AssignmentOperator.ADD) { - bin.operator = BinaryOperator.PLUS; - } else if (operator == AssignmentOperator.SUB) { - bin.operator = BinaryOperator.MINUS; - } else if (operator == AssignmentOperator.MUL) { - bin.operator = BinaryOperator.MUL; - } else if (operator == AssignmentOperator.DIV) { - bin.operator = BinaryOperator.DIV; - } else if (operator == AssignmentOperator.PERCENT) { - bin.operator = BinaryOperator.MOD; - } else if (operator == AssignmentOperator.SHIFT_LEFT) { - bin.operator = BinaryOperator.SHIFT_LEFT; - } else if (operator == AssignmentOperator.SHIFT_RIGHT) { - bin.operator = BinaryOperator.SHIFT_RIGHT; + switch (operator) { + case AssignmentOperator.BITWISE_OR: bin.operator = BinaryOperator.BITWISE_OR; break; + case AssignmentOperator.BITWISE_AND: bin.operator = BinaryOperator.BITWISE_AND; break; + case AssignmentOperator.BITWISE_XOR: bin.operator = BinaryOperator.BITWISE_XOR; break; + case AssignmentOperator.ADD: bin.operator = BinaryOperator.PLUS; break; + case AssignmentOperator.SUB: bin.operator = BinaryOperator.MINUS; break; + case AssignmentOperator.MUL: bin.operator = BinaryOperator.MUL; break; + case AssignmentOperator.DIV: bin.operator = BinaryOperator.DIV; break; + case AssignmentOperator.PERCENT: bin.operator = BinaryOperator.MOD; break; + case AssignmentOperator.SHIFT_LEFT: bin.operator = BinaryOperator.SHIFT_LEFT; break; + case AssignmentOperator.SHIFT_RIGHT: bin.operator = BinaryOperator.SHIFT_RIGHT; break; } right = bin; |