summaryrefslogtreecommitdiff
path: root/mesonbuild/scripts/depscan.py
diff options
context:
space:
mode:
authorDylan Baker <dylan@pnwbakers.com>2023-11-21 00:01:37 -0800
committerDylan Baker <dylan@pnwbakers.com>2024-03-29 13:06:54 -0700
commit3e9021a4c421d88501ee04aaaebb0e03f1f05540 (patch)
treecc593f0ebc0287cdba708d7c63e7ac3e0929c47f /mesonbuild/scripts/depscan.py
parent433117fc5a604e34a847b3480bbec15e59585b96 (diff)
downloadmeson-3e9021a4c421d88501ee04aaaebb0e03f1f05540.tar.gz
scripts/depscan: remove unnecessary function
This basically existed for an assert which we don't need, as mypy would catch that issue for us anyway. Removing the function entirely has some small performance advantages
Diffstat (limited to 'mesonbuild/scripts/depscan.py')
-rw-r--r--mesonbuild/scripts/depscan.py7
1 files changed, 1 insertions, 6 deletions
diff --git a/mesonbuild/scripts/depscan.py b/mesonbuild/scripts/depscan.py
index 79934fb5f..44e805447 100644
--- a/mesonbuild/scripts/depscan.py
+++ b/mesonbuild/scripts/depscan.py
@@ -107,11 +107,6 @@ class DependencyScanner:
self.provided_by[exported_module] = fname
self.exports[fname] = exported_module
- def objname_for(self, src: str) -> str:
- objname = self.target_data.source2object[src]
- assert isinstance(objname, str)
- return objname
-
def module_name_for(self, src: str, lang: Literal['cpp', 'fortran']) -> str:
if lang == 'fortran':
exported = self.exports[src]
@@ -131,7 +126,7 @@ class DependencyScanner:
with open(self.outfile, 'w', encoding='utf-8') as ofile:
ofile.write('ninja_dyndep_version = 1\n')
for src, lang in self.sources:
- objfilename = self.objname_for(src)
+ objfilename = self.target_data.source2object[src]
mods_and_submods_needed = []
module_files_generated = []
module_files_needed = []