Fix errors in CI. (#540)

* Add missing feature.

* Allow unused methods in test code.

* Fix format string.

* Fix tracing macro definition.

Author: jdm

html5ever/Cargo.toml

@@ -11,6 +11,9 @@ build = "build.rs"
 categories = [ "parser-implementations", "web-programming" ]
 edition = "2021"
 
+[features]
+trace_tokenizer = []
+
 [dependencies]
 log = "0.4"
 mac = "0.1"

html5ever/src/tokenizer/mod.rs

@@ -616,13 +616,13 @@ macro_rules! shorthand (
 
 // Tracing of tokenizer actions.  This adds significant bloat and compile time,
 // so it's behind a cfg flag.
-#[cfg(trace_tokenizer)]
+#[cfg(feature = "trace_tokenizer")]
 macro_rules! sh_trace ( ( $me:ident : $($cmds:tt)* ) => ({
-    trace!("  {:s}", stringify!($($cmds)*));
-    shorthand!($me:expr : $($cmds)*);
+    trace!("  {:?}", stringify!($($cmds)*));
+    shorthand!($me : $($cmds)*);
 }));
 
-#[cfg(not(trace_tokenizer))]
+#[cfg(not(feature = "trace_tokenizer"))]
 macro_rules! sh_trace ( ( $me:ident : $($cmds:tt)* ) => ( shorthand!($me: $($cmds)*) ) );
 
 // A little DSL for sequencing shorthand actions.

rcdom/tests/xml-tokenizer.rs

@@ -140,6 +140,7 @@ fn tokenize_xml(input: Vec<StrTendril>, opts: XmlTokenizerOpts) -> Vec<Token> {
     tok.sink.get_tokens()
 }
 
+#[allow(dead_code)]
 trait JsonExt: Sized {
     fn get_str(&self) -> String;
     fn get_tendril(&self) -> StrTendril;

xml5ever/Cargo.toml

@@ -13,6 +13,9 @@ exclude = ["xml5lib-tests/*"]
 categories = ["parser-implementations", "web-programming"]
 edition = "2021"
 
+[features]
+trace_tokenizer = []
+
 [dependencies]
 log = "0.4"
 mac = "0.1"

xml5ever/src/tokenizer/mod.rs

@@ -546,13 +546,13 @@ macro_rules! shorthand (
 
 // Tracing of tokenizer actions.  This adds significant bloat and compile time,
 // so it's behind a cfg flag.
-#[cfg(trace_tokenizer)]
+#[cfg(feature = "trace_tokenizer")]
 macro_rules! sh_trace ( ( $me:ident : $($cmds:tt)* ) => ({
-    debug!("  {:s}", stringify!($($cmds)*));
-    shorthand!($me:expr : $($cmds)*);
+    debug!("  {:?}", stringify!($($cmds)*));
+    shorthand!($me : $($cmds)*);
 }));
 
-#[cfg(not(trace_tokenizer))]
+#[cfg(not(feature = "trace_tokenizer"))]
 macro_rules! sh_trace ( ( $me:ident : $($cmds:tt)* ) => ( shorthand!($me: $($cmds)*) ) );
 
 // A little DSL for sequencing shorthand actions.