You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@avro.apache.org by "Ryan Skraba (Jira)" <ji...@apache.org> on 2021/09/03 13:14:00 UTC
[jira] [Resolved] (AVRO-3173) Rust: fix panic messages in test code
[ https://issues.apache.org/jira/browse/AVRO-3173?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Ryan Skraba resolved AVRO-3173.
-------------------------------
Resolution: Fixed
> Rust: fix panic messages in test code
> -------------------------------------
>
> Key: AVRO-3173
> URL: https://issues.apache.org/jira/browse/AVRO-3173
> Project: Apache Avro
> Issue Type: Improvement
> Components: rust
> Reporter: Martin Tzvetanov Grigorov
> Priority: Minor
> Labels: pull-request-available
> Fix For: 1.11.0
>
> Time Spent: 20m
> Remaining Estimate: 0h
>
> Running `cargo test` (Rust 1.53.0) produces the following warnings:
>
> {code:java}
> warning: panic message is not a string literal
> --> tests/schema.rs:1173:13
> |
> 1173 | e.to_string()
> | ^^^^^^^^^^^^^
> |
> = note: `#[warn(non_fmt_panic)]` on by default
> = note: this is no longer accepted in Rust 2021
> help: add a "{}" format string to Display the message
> |
> 1173 | "{}", e.to_string()
> | ^^^^^warning: 1 warning emitted
> {code}
>
> {code:java}
> warning: panic message is not a string literal
> --> tests/io.rs:105:13
> |
> 105 | format!("value {:?} does not validate schema: {}", value, raw_schema)
> | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> |
> = note: `#[warn(non_fmt_panic)]` on by default
> = note: this is no longer accepted in Rust 2021
> = note: the panic!() macro supports formatting, so there's no need for the format!() macro here
> help: remove the `format!(..)` macro call
> |
> 105 | "value {:?} does not validate schema: {}", value, raw_schema
> | -- --warning: 1 warning emitted{code}
--
This message was sent by Atlassian Jira
(v8.3.4#803005)