scons: Replace the SourceFile.filename property with attribute.
The SourceFile.filename property dynamically calculated the str()
conversion of self.tnode. Since self.tnode shouldn't be changed, it
doesn't seem useful to calculate that value over and over, especially
since it adds some extra indirection and magic to something that's
really pretty simple.
Change-Id: Ia0e1e8f4b0c019a026a08b5c2730d93c66de8190
Reviewed-on: https://gem5-review.googlesource.com/c/public/gem5/+/48123
Reviewed-by: Daniel Carvalho <odanrc@yahoo.com.br>
Maintainer: Bobby R. Bruce <bbruce@ucdavis.edu>
Tested-by: kokoro <noreply+kokoro@google.com>
diff --git a/src/SConscript b/src/SConscript
index 1565cd3..53df4f5 100644
--- a/src/SConscript
+++ b/src/SConscript
@@ -250,6 +250,7 @@
tnode = File(source)
self.tnode = tnode
+ self.filename = str(self.tnode)
self.snode = tnode.srcnode()
for base in type(self).__mro__:
@@ -275,10 +276,6 @@
return self.shared_objs[key]
@property
- def filename(self):
- return str(self.tnode)
-
- @property
def basename(self):
return basename(self.filename)
