stylo: Fix missing null terminators in error reporting code.

This was causing the patch in bug 1420026 to fail.
This commit is contained in:
Emilio Cobos Álvarez 2018-01-31 17:43:29 +01:00
parent c88dc51d03
commit d82c54bd30
No known key found for this signature in database
GPG key ID: 056B727BB9C1027C
2 changed files with 4 additions and 2 deletions

View file

@ -13,6 +13,7 @@ use style_traits::ParseError;
use stylesheets::UrlExtraData; use stylesheets::UrlExtraData;
/// Errors that can be encountered while parsing CSS. /// Errors that can be encountered while parsing CSS.
#[derive(Debug)]
pub enum ContextualParseError<'a> { pub enum ContextualParseError<'a> {
/// A property declaration was not recognized. /// A property declaration was not recognized.
UnsupportedPropertyDeclaration(&'a str, ParseError<'a>), UnsupportedPropertyDeclaration(&'a str, ParseError<'a>),

View file

@ -66,6 +66,7 @@ impl<'a> ErrorString<'a> {
} }
} }
#[derive(Debug)]
enum Action { enum Action {
Nothing, Nothing,
Skip, Skip,
@ -370,14 +371,14 @@ impl<'a> ErrorHelpers<'a> for ContextualParseError<'a> {
StyleParseErrorKind::ValueError( StyleParseErrorKind::ValueError(
ValueParseErrorKind::InvalidColor(..) ValueParseErrorKind::InvalidColor(..)
) )
) => (b"PEColorNotColor", Action::Nothing), ) => (b"PEColorNotColor\0", Action::Nothing),
_ => { _ => {
// Not the best error message, since we weren't parsing // Not the best error message, since we weren't parsing
// a declaration, just a value. But we don't produce // a declaration, just a value. But we don't produce
// UnsupportedValue errors other than InvalidColors // UnsupportedValue errors other than InvalidColors
// currently. // currently.
debug_assert!(false, "should use a more specific error message"); debug_assert!(false, "should use a more specific error message");
(b"PEDeclDropped", Action::Nothing) (b"PEDeclDropped\0", Action::Nothing)
} }
} }
} }