diff options
| author | Dylan Baker <dylan@pnwbakers.com> | 2023-11-21 00:01:37 -0800 |
|---|---|---|
| committer | Dylan Baker <dylan@pnwbakers.com> | 2024-03-29 13:06:54 -0700 |
| commit | 3e9021a4c421d88501ee04aaaebb0e03f1f05540 (patch) | |
| tree | cc593f0ebc0287cdba708d7c63e7ac3e0929c47f /mesonbuild/scripts/depscan.py | |
| parent | 433117fc5a604e34a847b3480bbec15e59585b96 (diff) | |
| download | meson-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.py | 7 |
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 = [] |
