From 5e54b29acf87eacf0a0255a9d3db8966de697fcf Mon Sep 17 00:00:00 2001 From: Rain Date: Sun, 17 Apr 2022 15:27:11 -0700 Subject: [PATCH] feat(spans): make SourceSpan implement Copy (#151) SourceSpan is just a pair of usizes, so it's cheap to copy. --- src/named_source.rs | 2 +- src/protocol.rs | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/named_source.rs b/src/named_source.rs index 06c8445..e6581bf 100644 --- a/src/named_source.rs +++ b/src/named_source.rs @@ -47,7 +47,7 @@ impl SourceCode for NamedSource { Ok(Box::new(MietteSpanContents::new_named( self.name.clone(), contents.data(), - contents.span().clone(), + *contents.span(), contents.line(), contents.column(), contents.line_count(), diff --git a/src/protocol.rs b/src/protocol.rs index 4b29f55..e829f66 100644 --- a/src/protocol.rs +++ b/src/protocol.rs @@ -345,7 +345,7 @@ impl<'a> SpanContents<'a> for MietteSpanContents<'a> { /** Span within a [`SourceCode`] with an associated message. */ -#[derive(Clone, Debug, Eq, PartialEq, Hash)] +#[derive(Clone, Copy, Debug, Eq, PartialEq, Hash)] pub struct SourceSpan { /// The start of the span. offset: SourceOffset,