Update if sequence in _check_all_source_locations
We can simplify the if flow in `_check_all_source_locations` by checking
if the field is a dataclass first.
diff --git a/compiler/front_end/module_ir_test.py b/compiler/front_end/module_ir_test.py
index dba5523..4eac8f1 100644
--- a/compiler/front_end/module_ir_test.py
+++ b/compiler/front_end/module_ir_test.py
@@ -4103,16 +4103,15 @@
if not proto.HasField(name):
continue
field_path = "{}{}".format(path, name)
- if spec.is_sequence:
- if spec.is_dataclass:
+ if spec.is_dataclass:
+ if spec.is_sequence:
index = 0
for i in getattr(proto, name):
item_path = "{}[{}]".format(field_path, index)
index += 1
errors.extend(
_check_all_source_locations(i, item_path, child_start, child_end))
- else:
- if spec.is_dataclass:
+ else:
errors.extend(_check_all_source_locations(getattr(proto, name),
field_path, child_start,
child_end))
