hohn/codeql
1
0
Fork 0
mirror of https://github.com/github/codeql.git synced 2026-04-26 01:05:15 +02:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #19938 from jketema/external

Browse Source 
C++: Remove unused `external_package` tables from the dbscheme
This commit is contained in:
Jeroen Ketema
2025-07-01 16:50:31 +02:00
committed by GitHub
parent 4c6c395b1b f3c5870d44
commit 7c2fd28585
9 changed files with 9765 additions and 321 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2428
cpp/downgrades/5491582ac8511726e12fae3e2399000f9201cd9a/old.dbscheme Normal file
View File

File diff suppressed because it is too large Load Diff

2451
cpp/downgrades/5491582ac8511726e12fae3e2399000f9201cd9a/semmlecode.cpp.dbscheme Normal file
View File

File diff suppressed because it is too large Load Diff

2
cpp/downgrades/5491582ac8511726e12fae3e2399000f9201cd9a/upgrade.properties Normal file
View File

@@ -0,0 +1,2 @@
description: Remove unused external_package tables from the dbscheme
compatibility: full

23
cpp/ql/lib/semmlecode.cpp.dbscheme
View File

@@ -217,29 +217,6 @@ diagnostics(
/*- C++ dbscheme -*/
/**
* Information about packages that provide code used during compilation.
* The `id` is just a unique identifier.
* The `namespace` is typically the name of the package manager that
* provided the package (e.g. "dpkg" or "yum").
* The `package_name` is the name of the package, and `version` is its
* version (as a string).
*/
external_packages(
unique int id: @external_package,
string namespace : string ref,
string package_name : string ref,
string version : string ref
);
/**
* Holds if File `fileid` was provided by package `package`.
*/
header_to_external_package(
int fileid : @file ref,
int package : @external_package ref
);
/*
* C++ dbscheme
*/

283
cpp/ql/lib/semmlecode.cpp.dbscheme.stats
View File

@@ -20,10 +20,6 @@
<k>@diagnostic</k>
<v>1483</v>
</e>
<e>
<k>@external_package</k>
<v>4</v>
</e>
<e>
<k>@location_default</k>
<v>46965948</v>
@@ -6192,285 +6188,6 @@
</dep>
</dependencies>
</relation>
<relation>
<name>external_packages</name>
<cardinality>4</cardinality>
<columnsizes>
<e>
<k>id</k>
<v>4</v>
</e>
<e>
<k>namespace</k>
<v>1</v>
</e>
<e>
<k>package_name</k>
<v>4</v>
</e>
<e>
<k>version</k>
<v>4</v>
</e>
</columnsizes>
<dependencies>
<dep>
<src>id</src>
<trg>namespace</trg>
<val>
<hist>
<budget>12</budget>
<bs>
<b>
<a>1</a>
<b>2</b>
<v>4</v>
</b>
</bs>
</hist>
</val>
</dep>
<dep>
<src>id</src>
<trg>package_name</trg>
<val>
<hist>
<budget>12</budget>
<bs>
<b>
<a>1</a>
<b>2</b>
<v>4</v>
</b>
</bs>
</hist>
</val>
</dep>
<dep>
<src>id</src>
<trg>version</trg>
<val>
<hist>
<budget>12</budget>
<bs>
<b>
<a>1</a>
<b>2</b>
<v>4</v>
</b>
</bs>
</hist>
</val>
</dep>
<dep>
<src>namespace</src>
<trg>id</trg>
<val>
<hist>
<budget>12</budget>
<bs>
<b>
<a>4</a>
<b>5</b>
<v>1</v>
</b>
</bs>
</hist>
</val>
</dep>
<dep>
<src>namespace</src>
<trg>package_name</trg>
<val>
<hist>
<budget>12</budget>
<bs>
<b>
<a>4</a>
<b>5</b>
<v>1</v>
</b>
</bs>
</hist>
</val>
</dep>
<dep>
<src>namespace</src>
<trg>version</trg>
<val>
<hist>
<budget>12</budget>
<bs>
<b>
<a>4</a>
<b>5</b>
<v>1</v>
</b>
</bs>
</hist>
</val>
</dep>
<dep>
<src>package_name</src>
<trg>id</trg>
<val>
<hist>
<budget>12</budget>
<bs>
<b>
<a>1</a>
<b>2</b>
<v>4</v>
</b>
</bs>
</hist>
</val>
</dep>
<dep>
<src>package_name</src>
<trg>namespace</trg>
<val>
<hist>
<budget>12</budget>
<bs>
<b>
<a>1</a>
<b>2</b>
<v>4</v>
</b>
</bs>
</hist>
</val>
</dep>
<dep>
<src>package_name</src>
<trg>version</trg>
<val>
<hist>
<budget>12</budget>
<bs>
<b>
<a>1</a>
<b>2</b>
<v>4</v>
</b>
</bs>
</hist>
</val>
</dep>
<dep>
<src>version</src>
<trg>id</trg>
<val>
<hist>
<budget>12</budget>
<bs>
<b>
<a>1</a>
<b>2</b>
<v>4</v>
</b>
</bs>
</hist>
</val>
</dep>
<dep>
<src>version</src>
<trg>namespace</trg>
<val>
<hist>
<budget>12</budget>
<bs>
<b>
<a>1</a>
<b>2</b>
<v>4</v>
</b>
</bs>
</hist>
</val>
</dep>
<dep>
<src>version</src>
<trg>package_name</trg>
<val>
<hist>
<budget>12</budget>
<bs>
<b>
<a>1</a>
<b>2</b>
<v>4</v>
</b>
</bs>
</hist>
</val>
</dep>
</dependencies>
</relation>
<relation>
<name>header_to_external_package</name>
<cardinality>92</cardinality>
<columnsizes>
<e>
<k>fileid</k>
<v>92</v>
</e>
<e>
<k>package</k>
<v>4</v>
</e>
</columnsizes>
<dependencies>
<dep>
<src>fileid</src>
<trg>package</trg>
<val>
<hist>
<budget>12</budget>
<bs>
<b>
<a>1</a>
<b>2</b>
<v>92</v>
</b>
</bs>
</hist>
</val>
</dep>
<dep>
<src>package</src>
<trg>fileid</trg>
<val>
<hist>
<budget>12</budget>
<bs>
<b>
<a>1</a>
<b>2</b>
<v>1</v>
</b>
<b>
<a>5</a>
<b>6</b>
<v>1</v>
</b>
<b>
<a>6</a>
<b>7</b>
<v>1</v>
</b>
<b>
<a>80</a>
<b>81</b>
<v>1</v>
</b>
</bs>
</hist>
</val>
</dep>
</dependencies>
</relation>
<relation>
<name>extractor_version</name>
<cardinality>124</cardinality>

2451
cpp/ql/lib/upgrades/827dbc206ea55377e032a8a934c8903fedc50fa0/old.dbscheme Normal file
View File

File diff suppressed because it is too large Load Diff

2428
cpp/ql/lib/upgrades/827dbc206ea55377e032a8a934c8903fedc50fa0/semmlecode.cpp.dbscheme Normal file
View File

File diff suppressed because it is too large Load Diff

4
cpp/ql/lib/upgrades/827dbc206ea55377e032a8a934c8903fedc50fa0/upgrade.properties Normal file
View File

@@ -0,0 +1,4 @@
description: Remove unused external_package tables from the dbscheme
compatibility: full
external_packages.rel: delete
header_to_external_package.rel: delete

16
cpp/ql/src/Metrics/Dependencies/ExternalDependencies.qll
View File

@@ -26,12 +26,6 @@ private newtype LibraryT =
LibraryTElement(LibraryElement lib, string name, string version) {
lib.getName() = name and
lib.getVersion() = version
} or
LibraryTExternalPackage(@external_package ep, string name, string version) {
exists(string package_name |
external_packages(ep, _, package_name, version) and
name = package_name
)
}
/**
@@ -41,10 +35,7 @@ class Library extends LibraryT {
string name;
string version;
Library() {
this = LibraryTElement(_, name, version) or
this = LibraryTExternalPackage(_, name, version)
}
Library() { this = LibraryTElement(_, name, version) }
string getName() { result = name }
@@ -63,11 +54,6 @@ class Library extends LibraryT {
this = LibraryTElement(lib, _, _) and
result = lib.getAFile()
)
or
exists(@external_package ep |
this = LibraryTExternalPackage(ep, _, _) and
header_to_external_package(unresolveElement(result), ep)
)
}
}