Skip to content

Commit

Permalink
Add derive macro implementation for diagnostic_source
Browse files Browse the repository at this point in the history
Signed-off-by: Matthias Beyer <[email protected]>
  • Loading branch information
matthiasbeyer committed May 5, 2022
1 parent e37fcc6 commit 9ed3ee3
Show file tree
Hide file tree
Showing 5 changed files with 123 additions and 1 deletion.
15 changes: 15 additions & 0 deletions miette-derive/src/diagnostic.rs
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ use syn::{punctuated::Punctuated, DeriveInput, Token};

use crate::code::Code;
use crate::diagnostic_arg::DiagnosticArg;
use crate::diagnostic_source::DiagnosticSource;
use crate::forward::{Forward, WhichFn};
use crate::help::Help;
use crate::label::Labels;
Expand Down Expand Up @@ -66,6 +67,7 @@ pub struct DiagnosticConcreteArgs {
pub url: Option<Url>,
pub forward: Option<Forward>,
pub related: Option<Related>,
pub diagnostic_source: Option<DiagnosticSource>,
}

impl DiagnosticConcreteArgs {
Expand All @@ -74,6 +76,7 @@ impl DiagnosticConcreteArgs {
let source_code = SourceCode::from_fields(fields)?;
let related = Related::from_fields(fields)?;
let help = Help::from_fields(fields)?;
let diagnostic_source = DiagnosticSource::from_fields(fields)?;
Ok(DiagnosticConcreteArgs {
code: None,
help,
Expand All @@ -83,6 +86,7 @@ impl DiagnosticConcreteArgs {
url: None,
forward: None,
source_code,
diagnostic_source,
})
}

Expand Down Expand Up @@ -283,6 +287,8 @@ impl Diagnostic {
let source_code_method = forward.gen_struct_method(WhichFn::SourceCode);
let severity_method = forward.gen_struct_method(WhichFn::Severity);
let related_method = forward.gen_struct_method(WhichFn::Related);
let diagnostic_source_method =
forward.gen_struct_method(WhichFn::DiagnosticSource);

quote! {
impl #impl_generics miette::Diagnostic for #ident #ty_generics #where_clause {
Expand All @@ -293,6 +299,7 @@ impl Diagnostic {
#severity_method
#source_code_method
#related_method
#diagnostic_source_method
}
}
}
Expand Down Expand Up @@ -338,6 +345,11 @@ impl Diagnostic {
.as_ref()
.and_then(|x| x.gen_struct(fields))
.or_else(|| forward(WhichFn::SourceCode));
let diagnostic_source = concrete
.diagnostic_source
.as_ref()
.and_then(|x| x.gen_struct())
.or_else(|| forward(WhichFn::DiagnosticSource));
quote! {
impl #impl_generics miette::Diagnostic for #ident #ty_generics #where_clause {
#code_body
Expand All @@ -347,6 +359,7 @@ impl Diagnostic {
#url_body
#labels_body
#src_body
#diagnostic_source
}
}
}
Expand All @@ -365,6 +378,7 @@ impl Diagnostic {
let src_body = SourceCode::gen_enum(variants);
let rel_body = Related::gen_enum(variants);
let url_body = Url::gen_enum(ident, variants);
let diagnostic_source_body = DiagnosticSource::gen_enum(variants);
quote! {
impl #impl_generics miette::Diagnostic for #ident #ty_generics #where_clause {
#code_body
Expand All @@ -374,6 +388,7 @@ impl Diagnostic {
#src_body
#rel_body
#url_body
#diagnostic_source_body
}
}
}
Expand Down
78 changes: 78 additions & 0 deletions miette-derive/src/diagnostic_source.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,78 @@
use proc_macro2::TokenStream;
use quote::quote;
use syn::spanned::Spanned;

use crate::forward::WhichFn;
use crate::{
diagnostic::{DiagnosticConcreteArgs, DiagnosticDef},
utils::{display_pat_members, gen_all_variants_with},
};

pub struct DiagnosticSource(syn::Member);

impl DiagnosticSource {
pub(crate) fn from_fields(fields: &syn::Fields) -> syn::Result<Option<Self>> {
match fields {
syn::Fields::Named(named) => Self::from_fields_vec(named.named.iter().collect()),
syn::Fields::Unnamed(unnamed) => {
Self::from_fields_vec(unnamed.unnamed.iter().collect())
}
syn::Fields::Unit => Ok(None),
}
}

fn from_fields_vec(fields: Vec<&syn::Field>) -> syn::Result<Option<Self>> {
for (i, field) in fields.iter().enumerate() {
for attr in &field.attrs {
if attr.path.is_ident("diagnostic_source") {
let diagnostic_source = if let Some(ident) = field.ident.clone() {
syn::Member::Named(ident)
} else {
syn::Member::Unnamed(syn::Index {
index: i as u32,
span: field.span(),
})
};
return Ok(Some(DiagnosticSource(diagnostic_source)));
}
}
}
Ok(None)
}

pub(crate) fn gen_enum(variants: &[DiagnosticDef]) -> Option<TokenStream> {
gen_all_variants_with(
variants,
WhichFn::DiagnosticSource,
|ident,
fields,
DiagnosticConcreteArgs {
diagnostic_source, ..
}| {
let (display_pat, _display_members) = display_pat_members(fields);
diagnostic_source.as_ref().map(|diagnostic_source| {
let rel = match &diagnostic_source.0 {
syn::Member::Named(ident) => ident.clone(),
syn::Member::Unnamed(syn::Index { index, .. }) => {
quote::format_ident!("_{}", index)
}
};
quote! {
Self::#ident #display_pat => {
std::option::Option::Some(#rel.as_ref())
}
}
})
},
)
}

pub(crate) fn gen_struct(&self) -> Option<TokenStream> {
let rel = &self.0;
Some(quote! {
fn diagnostic_source<'a>(&'a self) -> std::option::Option<&'a dyn miette::Diagnostic> {
std::option::Option::Some(&self.#rel)
}
})
}
}
5 changes: 5 additions & 0 deletions miette-derive/src/forward.rs
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@ pub enum WhichFn {
Labels,
SourceCode,
Related,
DiagnosticSource,
}

impl WhichFn {
Expand All @@ -50,6 +51,7 @@ impl WhichFn {
Self::Labels => quote! { labels() },
Self::SourceCode => quote! { source_code() },
Self::Related => quote! { related() },
Self::DiagnosticSource => quote! { diagnostic_source() },
}
}

Expand All @@ -76,6 +78,9 @@ impl WhichFn {
Self::SourceCode => quote! {
fn source_code(&self) -> std::option::Option<&dyn miette::SourceCode>
},
Self::DiagnosticSource => quote! {
fn diagnostic_source(&self) -> std::option::Option<&dyn miette::Diagnostic>
},
}
}

Expand Down
6 changes: 5 additions & 1 deletion miette-derive/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ use diagnostic::Diagnostic;
mod code;
mod diagnostic;
mod diagnostic_arg;
mod diagnostic_source;
mod fmt;
mod forward;
mod help;
Expand All @@ -16,7 +17,10 @@ mod source_code;
mod url;
mod utils;

#[proc_macro_derive(Diagnostic, attributes(diagnostic, source_code, label, related, help))]
#[proc_macro_derive(
Diagnostic,
attributes(diagnostic, source_code, label, related, help, diagnostic_source)
)]
pub fn derive_diagnostic(input: proc_macro::TokenStream) -> proc_macro::TokenStream {
let input = parse_macro_input!(input as DeriveInput);
let cmd = match Diagnostic::from_derive_input(input) {
Expand Down
20 changes: 20 additions & 0 deletions tests/test_diagnostic_source_macro.rs
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
use miette::Diagnostic;

#[derive(Debug, miette::Diagnostic, thiserror::Error)]
#[error("AnErr")]
struct AnErr;

#[derive(Debug, miette::Diagnostic, thiserror::Error)]
#[error("TestError")]
struct TestError {
#[diagnostic_source]
asdf_inner_foo: AnErr,
}

#[test]
fn test_diagnostic_source() {
let error = TestError {
asdf_inner_foo: AnErr,
};
assert!(error.diagnostic_source().is_some());
}

0 comments on commit 9ed3ee3

Please sign in to comment.