cleaning

Author: ducky64

edg/core/Binding.py

@@ -111,7 +111,7 @@ class InitParamBinding(ParamBinding):
   Can optionally take a value, which would have a binding in the parent's scope."""
   def __init__(self, parent: ParamParentTypes, value: Optional[ConstraintExpr] = None):
     super().__init__(parent)
-    self._value = value
+    self.value = value
 
 
 class LiteralBinding(Binding):

edg/core/HierarchyBlock.py

@@ -277,9 +277,9 @@ def _populate_def_proto_block_base(self, pb: edgir.HierarchyBlock) -> edgir.Hier
 
     # generate param defaults
     for param_name, param in self._parameters.items():
-      if isinstance(param.binding, InitParamBinding) and param.binding._value is not None:
+      if isinstance(param.binding, InitParamBinding) and param.binding.value is not None:
         # default values can't depend on anything so the ref_map is empty
-        pb.param_defaults[param_name].CopyFrom(param.binding._value._expr_to_proto(IdentityDict()))
+        pb.param_defaults[param_name].CopyFrom(param.binding.value._expr_to_proto(IdentityDict()))
 
     return pb
 
@@ -384,9 +384,9 @@ def _populate_def_proto_hierarchy(self, pb: edgir.HierarchyBlock) -> edgir.Hiera
       for mixin in block._mixins:
         all_block_params.update(mixin._parameters.items())
       for (block_param_name, block_param) in all_block_params.items():
-        if isinstance(block_param.binding, InitParamBinding) and block_param.binding._value is not None:
+        if isinstance(block_param.binding, InitParamBinding) and block_param.binding.value is not None:
           edgir.add_pair(pb.constraints, f'(init){block_name}.{block_param_name}').CopyFrom(  # TODO better name
-            AssignBinding.make_assign(block_param, block_param.binding._value, ref_map)
+            AssignBinding.make_assign(block_param, block_param.binding.value, ref_map)
           )
 
     return pb