Skip to content

Commit b40b276

Browse files
psteinroeclaude
andauthored
fix: clippy issues (#622)
fixes minor clippy issues from previous prs --------- Co-authored-by: Claude <noreply@anthropic.com>
1 parent 813624a commit b40b276

File tree

1 file changed

+7
-23
lines changed

1 file changed

+7
-23
lines changed

crates/pgls_completions/src/relevance/scoring.rs

Lines changed: 7 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -73,14 +73,13 @@ impl CompletionScore<'_> {
7373
self.get_table_name()
7474
.map(|t| format!("{t}.{name}"))
7575
.unwrap_or(name.clone())
76-
} else if self.get_table_name().is_some_and(|t| t == qualifier) {
77-
name.clone()
78-
} else if ctx
79-
.get_mentioned_table_for_alias(qualifier)
80-
.is_some_and(|alias_tbl| {
81-
self.get_table_name()
82-
.is_some_and(|item_tbl| alias_tbl == item_tbl)
83-
})
76+
} else if self.get_table_name().is_some_and(|t| t == qualifier)
77+
|| ctx
78+
.get_mentioned_table_for_alias(qualifier)
79+
.is_some_and(|alias_tbl| {
80+
self.get_table_name()
81+
.is_some_and(|item_tbl| alias_tbl == item_tbl)
82+
})
8483
{
8584
name.clone()
8685
} else {
@@ -309,21 +308,6 @@ impl CompletionScore<'_> {
309308
}
310309
}
311310

312-
fn get_fully_qualified_name(&self) -> String {
313-
match self.data {
314-
CompletionRelevanceData::Schema(s) => s.name.clone(),
315-
CompletionRelevanceData::Column(c) => {
316-
format!("{}.{}.{}", c.schema_name, c.table_name, c.name)
317-
}
318-
CompletionRelevanceData::Table(t) => format!("{}.{}", t.schema, t.name),
319-
CompletionRelevanceData::Function(f) => format!("{}.{}", f.schema, f.name),
320-
CompletionRelevanceData::Policy(p) => {
321-
format!("{}.{}.{}", p.schema_name, p.table_name, p.name)
322-
}
323-
CompletionRelevanceData::Role(r) => r.name.clone(),
324-
}
325-
}
326-
327311
fn get_table_name(&self) -> Option<&str> {
328312
match self.data {
329313
CompletionRelevanceData::Column(c) => Some(c.table_name.as_str()),

0 commit comments

Comments
 (0)