summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBenjamin Schubert <ben.c.schubert@gmail.com>2019-07-12 10:05:45 +0100
committerBenjamin Schubert <ben.c.schubert@gmail.com>2019-07-12 10:09:01 +0100
commit68238e47306d4ae6255bf45abde76ab2d7190168 (patch)
tree9542ca8eeea0af4875baf2bfefd36155c8168cfb
parent8303271fd221477a75d6f659f65d6a1d7a538885 (diff)
downloadbuildstream-68238e47306d4ae6255bf45abde76ab2d7190168.tar.gz
node: Mark `MappingNode._get` as private
-rw-r--r--src/buildstream/node.pxd2
-rw-r--r--src/buildstream/node.pyx34
2 files changed, 18 insertions, 18 deletions
diff --git a/src/buildstream/node.pxd b/src/buildstream/node.pxd
index 3069b2fc9..19f0a223e 100644
--- a/src/buildstream/node.pxd
+++ b/src/buildstream/node.pxd
@@ -66,10 +66,10 @@ cdef class MappingNode(Node):
# Protected Methods
cdef void _compose_on_composite_dict(self, MappingNode target)
cdef void _compose_on_list(self, SequenceNode target)
- cdef Node _get(self, str key, default, default_constructor)
# Private Methods
cdef void __composite(self, MappingNode target, list path=*) except *
+ cdef Node _get(self, str key, default, default_constructor)
cdef class ScalarNode(Node):
diff --git a/src/buildstream/node.pyx b/src/buildstream/node.pyx
index c14b26aca..19054efdc 100644
--- a/src/buildstream/node.pyx
+++ b/src/buildstream/node.pyx
@@ -550,23 +550,6 @@ cdef class MappingNode(Node):
else:
target.value["(>)"] = suffix
- cdef Node _get(self, str key, object default, object default_constructor):
- value = self.value.get(key, _sentinel)
-
- if value is _sentinel:
- if default is _sentinel:
- provenance = self.get_provenance()
- raise LoadError(LoadErrorReason.INVALID_DATA,
- "{}: Dictionary did not contain expected key '{}'".format(provenance, key))
-
- if default is None:
- value = None
- else:
- value = default_constructor.__new__(
- default_constructor, _SYNTHETIC_FILE_INDEX, 0, next_synthetic_counter(), default)
-
- return value
-
cdef bint _is_composite_list(self) except *:
cdef bint has_directives = False
cdef bint has_keys = False
@@ -614,6 +597,23 @@ cdef class MappingNode(Node):
value._compose_on(key, target, path)
path.pop()
+ cdef Node _get(self, str key, object default, object default_constructor):
+ value = self.value.get(key, _sentinel)
+
+ if value is _sentinel:
+ if default is _sentinel:
+ provenance = self.get_provenance()
+ raise LoadError(LoadErrorReason.INVALID_DATA,
+ "{}: Dictionary did not contain expected key '{}'".format(provenance, key))
+
+ if default is None:
+ value = None
+ else:
+ value = default_constructor.__new__(
+ default_constructor, _SYNTHETIC_FILE_INDEX, 0, next_synthetic_counter(), default)
+
+ return value
+
cdef class SequenceNode(Node):
def __cinit__(self, int file_index, int line, int column, list value):