Skip to content

Commit

Permalink
feat(error): diagnostic-ify MietteError
Browse files Browse the repository at this point in the history
  • Loading branch information
zkat committed Aug 22, 2021
1 parent adf0bc9 commit e980b72
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 4 deletions.
21 changes: 18 additions & 3 deletions src/error.rs
Original file line number Diff line number Diff line change
Expand Up @@ -2,15 +2,30 @@ use std::io;

use thiserror::Error;

use crate::{self as miette, Diagnostic};

/**
Error enum for miette. Used by certain operations in the protocol.
*/
#[derive(Debug, Error)]
#[derive(Debug, Diagnostic, Error)]
pub enum MietteError {
/// Wrapper around [std::io::Error]. This is returned when something went
/// wrong while reading a [crate::Source].
#[error(transparent)]
#[diagnostic(code(miette::io_error))]
IoError(#[from] io::Error),

/// Returned when a [crate::SourceSpan] extends beyond the bounds of a given [crate::Source].
#[error("The given offset is outside the bounds of its Source")]
#[diagnostic(
code(miette::span_out_of_bounds),
help("Double-check your spans. Do you have an off-by-one error?")
)]
OutOfBounds,
#[error("Failed to install reporter hook")]
ReporterInstallFailed,

/// Returned when installing a [crate::DiagnosticReportPrinter] failed.
/// Typically, this will be because [crate::set_printer] was called twice.
#[error("Failed to install DiagnosticReportPrinter")]
#[diagnostic(code(miette::set_printer_failed))]
SetPrinterFailure,
}
2 changes: 1 addition & 1 deletion src/printer/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ pub fn set_reporter(
) -> Result<(), MietteError> {
REPORTER
.set(Box::new(reporter))
.map_err(|_| MietteError::ReporterInstallFailed)
.map_err(|_| MietteError::SetPrinterFailure)
}

/// Used by [DiagnosticReport] to fetch the reporter that will be used to
Expand Down

0 comments on commit e980b72

Please sign in to comment.