From 531722229bf4d68e318fa9718201edd2db36d2a1 Mon Sep 17 00:00:00 2001 From: Xavier Claessens Date: Sun, 13 May 2018 10:36:58 -0400 Subject: UserUmaskOption: Implement validate_value instead of set_value set_value() is implemented by the base class, subclasses only need to override validate_value(). --- mesonbuild/coredata.py | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/mesonbuild/coredata.py b/mesonbuild/coredata.py index 24580bc..228b6e1 100644 --- a/mesonbuild/coredata.py +++ b/mesonbuild/coredata.py @@ -109,11 +109,10 @@ class UserUmaskOption(UserIntegerOption): def __init__(self, name, description, value, yielding=None): super().__init__(name, description, 0, 0o777, value, yielding) - def set_value(self, newvalue): - if newvalue is None or newvalue == 'preserve': - self.value = None - else: - super().set_value(newvalue) + def validate_value(self, value): + if value is None or value == 'preserve': + return None + return super().validate_value(value) def toint(self, valuestring): try: -- cgit v1.1