diff options
author | Daniel Kolesa <d.kolesa@samsung.com> | 2019-05-26 17:25:15 +0200 |
---|---|---|
committer | Daniel Kolesa <d.kolesa@samsung.com> | 2019-05-26 17:25:15 +0200 |
commit | d0fed247bbb18933b3d5c086126ac0f7b8d05e76 (patch) | |
tree | 875995d0b38022dcad4fac08a4d454254a6648bc /src/scripts | |
parent | 38d7d33354ce4d13d70f707498c267b120bf2e5a (diff) |
eolian: remove param @nonull
This has been deprecated for a while and is not strictly necessary
- as a part of an effort to stabilize Eolian, remove this. Eolian
will eventually gain support for versioning and use a reversed
behavior (i.e. no NULL by default), but the API it wlll use for
that will be very different. Features can always be added, it's
much harder to drop them.
@feature
Diffstat (limited to 'src/scripts')
-rw-r--r-- | src/scripts/pyolian/eolian.py | 4 | ||||
-rw-r--r-- | src/scripts/pyolian/eolian_lib.py | 4 | ||||
-rwxr-xr-x | src/scripts/pyolian/test_eolian.py | 1 |
3 files changed, 0 insertions, 9 deletions
diff --git a/src/scripts/pyolian/eolian.py b/src/scripts/pyolian/eolian.py index bd92075565..216c94ed8a 100644 --- a/src/scripts/pyolian/eolian.py +++ b/src/scripts/pyolian/eolian.py | |||
@@ -975,10 +975,6 @@ class Function_Parameter(Object): | |||
975 | return Documentation(c_doc) if c_doc else None | 975 | return Documentation(c_doc) if c_doc else None |
976 | 976 | ||
977 | @cached_property | 977 | @cached_property |
978 | def is_nonull(self): | ||
979 | return bool(lib.eolian_parameter_is_nonull(self)) | ||
980 | |||
981 | @cached_property | ||
982 | def is_optional(self): | 978 | def is_optional(self): |
983 | return bool(lib.eolian_parameter_is_optional(self)) | 979 | return bool(lib.eolian_parameter_is_optional(self)) |
984 | 980 | ||
diff --git a/src/scripts/pyolian/eolian_lib.py b/src/scripts/pyolian/eolian_lib.py index 4fe2dc54fb..c278442c75 100644 --- a/src/scripts/pyolian/eolian_lib.py +++ b/src/scripts/pyolian/eolian_lib.py | |||
@@ -399,10 +399,6 @@ lib.eolian_parameter_default_value_get.restype = c_void_p | |||
399 | lib.eolian_parameter_documentation_get.argtypes = (c_void_p,) | 399 | lib.eolian_parameter_documentation_get.argtypes = (c_void_p,) |
400 | lib.eolian_parameter_documentation_get.restype = c_void_p | 400 | lib.eolian_parameter_documentation_get.restype = c_void_p |
401 | 401 | ||
402 | # EAPI Eina_Bool eolian_parameter_is_nonull(const Eolian_Function_Parameter *param_desc); | ||
403 | lib.eolian_parameter_is_nonull.argtypes = (c_void_p,) | ||
404 | lib.eolian_parameter_is_nonull.restype = c_bool | ||
405 | |||
406 | # EAPI Eina_Bool eolian_parameter_is_optional(const Eolian_Function_Parameter *param_desc); | 402 | # EAPI Eina_Bool eolian_parameter_is_optional(const Eolian_Function_Parameter *param_desc); |
407 | lib.eolian_parameter_is_optional.argtypes = (c_void_p,) | 403 | lib.eolian_parameter_is_optional.argtypes = (c_void_p,) |
408 | lib.eolian_parameter_is_optional.restype = c_bool | 404 | lib.eolian_parameter_is_optional.restype = c_bool |
diff --git a/src/scripts/pyolian/test_eolian.py b/src/scripts/pyolian/test_eolian.py index 2a59349386..df7740bc19 100755 --- a/src/scripts/pyolian/test_eolian.py +++ b/src/scripts/pyolian/test_eolian.py | |||
@@ -380,7 +380,6 @@ class TestEolianFunction(unittest.TestCase): | |||
380 | self.assertEqual(p.direction, eolian.Eolian_Parameter_Dir.IN) | 380 | self.assertEqual(p.direction, eolian.Eolian_Parameter_Dir.IN) |
381 | self.assertEqual(p.name, 'add') | 381 | self.assertEqual(p.name, 'add') |
382 | self.assertIsNone(p.default_value) | 382 | self.assertIsNone(p.default_value) |
383 | self.assertFalse(p.is_nonull) # TODO is correct ?? 'add' can be null? | ||
384 | self.assertFalse(p.is_optional) | 383 | self.assertFalse(p.is_optional) |
385 | self.assertEqual(p.type.name, 'double') | 384 | self.assertEqual(p.type.name, 'double') |
386 | self.assertIsInstance(p.documentation, eolian.Documentation) | 385 | self.assertIsInstance(p.documentation, eolian.Documentation) |