hohn/codeql
1
0
Fork 0
mirror of https://github.com/github/codeql.git synced 2025-12-16 16:53:25 +01:00
Code Issues Packages Projects Releases Wiki Activity

Rust: silence warning by removing unused generated function

Browse Source
This commit is contained in:
Paolo Tranquilli
2025-03-21 10:59:39 +01:00
parent 7d312feffe
commit 8f8fe2f8b6
2 changed files with 8 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
rust/ast-generator/src/main.rs
View File

@@ -465,8 +465,12 @@ struct ExtractorInfo {
nodes: Vec<ExtractorNodeInfo>,
}
fn enum_to_extractor_info(node: &AstEnumSrc) -> ExtractorEnumInfo {
ExtractorEnumInfo {
fn enum_to_extractor_info(node: &AstEnumSrc) -> Option<ExtractorEnumInfo> {
if node.name == "VariantDef" {
// currently defined but unused
return None;
}
Some(ExtractorEnumInfo {
name: class_name(&node.name),
snake_case_name: to_lower_snake_case(&node.name),
ast_name: node.name.clone(),
@@ -478,7 +482,7 @@ fn enum_to_extractor_info(node: &AstEnumSrc) -> ExtractorEnumInfo {
snake_case_name: to_lower_snake_case(v),
})
.collect(),
}
})
}
fn field_info_to_extractor_info(node: &AstNodeSrc, field: &FieldInfo) -> ExtractorNodeFieldInfo {
@@ -528,7 +532,7 @@ fn node_to_extractor_info(node: &AstNodeSrc) -> ExtractorNodeInfo {
fn write_extractor(grammar: &AstSrc) -> mustache::Result<String> {
let extractor_info = ExtractorInfo {
enums: grammar.enums.iter().map(enum_to_extractor_info).collect(),
enums: grammar.enums.iter().filter_map(enum_to_extractor_info).collect(),
nodes: grammar.nodes.iter().map(node_to_extractor_info).collect(),
};
let template = mustache::compile_str(include_str!("templates/extractor.mustache"))?;

8
rust/extractor/src/translate/generated.rs generated
View File

@@ -176,14 +176,6 @@ impl Translator<'_> {
}
}
pub(crate) fn emit_variant_def(&mut self, node: ast::VariantDef) -> Option<Label<generated::VariantDef>> {
match node {
ast::VariantDef::Struct(inner) => self.emit_struct(inner).map(Into::into),
ast::VariantDef::Union(inner) => self.emit_union(inner).map(Into::into),
ast::VariantDef::Variant(inner) => self.emit_variant(inner).map(Into::into),
}
}
pub(crate) fn emit_item(&mut self, node: ast::Item) -> Option<Label<generated::Item>> {
match node {
ast::Item::Const(inner) => self.emit_const(inner).map(Into::into),