Implement FromStr for enum Edition

This commit is contained in:
Metabaron 2019-11-11 23:16:59 +01:00
parent f5e1b0f97c
commit b69738590c
3 changed files with 14 additions and 8 deletions

View file

@ -1,5 +1,6 @@
//! FIXME: write short doc here
use std::str::FromStr;
use std::sync::Arc;
use ra_cfg::CfgOptions;
@ -164,7 +165,7 @@ fn parse_meta(meta: &str) -> ParsedMeta {
match key {
"crate" => krate = Some(value.to_string()),
"deps" => deps = value.split(',').map(|it| it.to_string()).collect(),
"edition" => edition = Edition::from_string(&value),
"edition" => edition = Edition::from_str(&value).unwrap(),
"cfg" => {
for key in value.split(',') {
match split1(key, '=') {

View file

@ -13,6 +13,7 @@ use ra_syntax::SmolStr;
use rustc_hash::FxHashSet;
use crate::{RelativePath, RelativePathBuf};
use std::str::FromStr;
/// `FileId` is an integer which uniquely identifies a file. File paths are
/// messy and system-dependent, so most of the code should work directly with
@ -97,12 +98,13 @@ pub enum Edition {
Edition2015,
}
impl Edition {
//FIXME: replace with FromStr with proper error handling
pub fn from_string(s: &str) -> Edition {
impl FromStr for Edition {
type Err = String;
fn from_str(s: &str) -> Result<Self, Self::Err> {
match s {
"2015" => Edition::Edition2015,
"2018" | _ => Edition::Edition2018,
"2015" => Ok(Edition::Edition2015),
"2018" => Ok(Edition::Edition2018),
_ => Err(format! {"unknown edition: {}" , s}),
}
}
}