Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Remove even more instances of @ts-expect-error from search.js #138913

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 16 commits into from
Apr 2, 2025
Merged

Remove even more instances of @ts-expect-error from search.js #138913

Changes from 1 commit
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
c123adf
rustdoc js: add nonundef and use it to remove a ts-expect-error
lolbinarycat Mar 24, 2025
ccd95ac
search.js: improve typechecking by avoiding Map.has
lolbinarycat Mar 24, 2025
e9a5470
search.js: refactor handling of rawPaths in buildIndex
lolbinarycat Mar 24, 2025
49bf6ca
search.js: add undef2null and eliminate more @ts-expect-error
lolbinarycat Mar 24, 2025
1af16cd
search.js(query parser): rethrow error if it isn't a string array
lolbinarycat Mar 24, 2025
8af647c
search.js: remove another Map.has() and @ts-expect-error
lolbinarycat Mar 24, 2025
714be45
search.js: refactor transformResults
lolbinarycat Mar 24, 2025
ca514c1
search.js: fix signature of pushText
lolbinarycat Mar 24, 2025
7f14198
search.js: use @type instead of @ts-expect-error
lolbinarycat Mar 24, 2025
3245e7b
search.js: fix return type of unifyFunctionTypes
lolbinarycat Mar 24, 2025
4a00a84
search.js: give type to unifiedGenericsMgens
lolbinarycat Mar 24, 2025
523f413
search.js: give type annotation to newSolutions
lolbinarycat Mar 24, 2025
1b8b09a
search.js: fix whitespace
lolbinarycat Mar 25, 2025
74b7599
satisfy eslint
lolbinarycat Mar 27, 2025
65624d9
tester.js: ignore displayTypeSignature if it is null
lolbinarycat Mar 27, 2025
d487087
search.js: revert usage of nonundef for now (not available under test)
lolbinarycat Mar 27, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
search.js: improve typechecking by avoiding Map.has
  • Loading branch information
@lolbinarycat
lolbinarycat committed Mar 24, 2025
commit ccd95ac7fde43d6a0380895086efca3a75e1ac35
21 changes: 8 additions & 13 deletions src/librustdoc/html/static/js/search.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1310,10 +1310,9 @@ class NameTrie {
let sste;
if (substart >= 2) {
const tail = name.substring(substart - 2, substart + 1);
if (tailTable.has(tail)) {
// it's not undefined
// @ts-expect-error
sste = tailTable.get(tail);
const entry = tailTable.get(tail);
if (entry !== undefined) {
sste = entry;
} else {
sste = [];
tailTable.set(tail, sste);
Expand All @@ -1337,10 +1336,9 @@ class NameTrie {
new Lev1TParametricDescription(name.length);
this.searchLev(name, 0, levParams, results);
const tail = name.substring(0, 3);
if (tailTable.has(tail)) {
// it's not undefined
// @ts-expect-error
for (const entry of tailTable.get(tail)) {
const list = tailTable.get(tail);
if (list !== undefined) {
for (const entry of list) {
entry.searchSubstringPrefix(name, 3, results);
}
}
Expand Down Expand Up @@ -1599,10 +1597,8 @@ class DocSearch {
return null;
}

if (this.typeNameIdMap.has(name)) {
/** @type {{id: number, assocOnly: boolean}} */
// @ts-expect-error
const obj = this.typeNameIdMap.get(name);
const obj = this.typeNameIdMap.get(name);
if (obj !== undefined) {
obj.assocOnly = !!(isAssocType && obj.assocOnly);
return obj.id;
} else {
Expand Down Expand Up @@ -2145,7 +2141,6 @@ class DocSearch {
const name = elem[1];
let path = null;
if (elem.length > 2 && elem[2] !== null) {
// @ts-expect-error
path = itemPaths.has(elem[2]) ? itemPaths.get(elem[2]) : lastPath;
lastPath = path;
}
Expand Down