Merge pull request #2482 from devitocodes/datasymbol

dsl: Add ArgProvider inheritance to AbstractSymbol

devito/types/basic.py

@@ -526,7 +526,7 @@ def __new__(cls, *args, **kwargs):
     __hash__ = Cached.__hash__
 
 
-class DataSymbol(AbstractSymbol, Uncached):
+class DataSymbol(AbstractSymbol, Uncached, ArgProvider):
 
     """
     A unique scalar symbol that carries data.

devito/types/constant.py

@@ -2,13 +2,12 @@
 
 from devito.exceptions import InvalidArgument
 from devito.logger import warning
-from devito.types.args import ArgProvider
 from devito.types.basic import DataSymbol
 
 __all__ = ['Constant']
 
 
-class Constant(DataSymbol, ArgProvider):
+class Constant(DataSymbol):
 
     """
     Symbol representing a constant, scalar value in symbolic equations.

devito/types/dimension.py

@@ -570,9 +570,6 @@ def local(self):
     def value(self):
         return self._value
 
-    def _arg_check(self, *args, **kwargs):
-        pass
-
     def _arg_values(self, grid=None, **kwargs):
         # Allow override of thickness values to disable BCs
         # However, arguments from the user are considered global
@@ -597,12 +594,6 @@ def _arg_values(self, grid=None, **kwargs):
 
         return {self.name: tkn}
 
-    def _arg_finalize(self, *args, **kwargs):
-        return {}
-
-    def _arg_apply(self, *args, **kwargs):
-        pass
-
 
 class AbstractSubDimension(DerivedDimension):