This commit is contained in:
Folkert 2021-02-20 22:28:01 +01:00
parent b098d13407
commit 232c8652d5
2 changed files with 12 additions and 10 deletions

View file

@ -705,6 +705,7 @@ fn x86_64_generic_setup_stack<'a>(
} }
#[inline(always)] #[inline(always)]
#[allow(clippy::unnecessary_wraps)]
fn x86_64_generic_cleanup_stack<'a>( fn x86_64_generic_cleanup_stack<'a>(
buf: &mut Vec<'a, u8>, buf: &mut Vec<'a, u8>,
saved_regs: &[X86_64GeneralReg], saved_regs: &[X86_64GeneralReg],

View file

@ -2212,7 +2212,7 @@ fn load_pkg_config<'a>(
&header, &header,
pkg_module_timing, pkg_module_timing,
) )
.map(|x| x.1)?; .1;
let effects_module_msg = fabricate_effects_module( let effects_module_msg = fabricate_effects_module(
arena, arena,
@ -2223,7 +2223,7 @@ fn load_pkg_config<'a>(
header, header,
effect_module_timing, effect_module_timing,
) )
.map(|x| x.1)?; .1;
Ok(Msg::Many(vec![effects_module_msg, pkg_config_module_msg])) Ok(Msg::Many(vec![effects_module_msg, pkg_config_module_msg]))
} }
@ -2465,7 +2465,7 @@ fn parse_header<'a>(
}, },
} }
} }
Ok((_, ast::Module::Platform { header }, _parse_state)) => fabricate_effects_module( Ok((_, ast::Module::Platform { header }, _parse_state)) => Ok(fabricate_effects_module(
arena, arena,
&"", &"",
module_ids, module_ids,
@ -2473,7 +2473,7 @@ fn parse_header<'a>(
mode, mode,
header, header,
module_timing, module_timing,
), )),
Err((_, fail, _)) => Err(LoadingProblem::ParsingFailed( Err((_, fail, _)) => Err(LoadingProblem::ParsingFailed(
fail.into_parse_problem(filename, src_bytes), fail.into_parse_problem(filename, src_bytes),
)), )),
@ -3093,11 +3093,11 @@ fn fabricate_pkg_config_module<'a>(
ident_ids_by_module: Arc<Mutex<MutMap<ModuleId, IdentIds>>>, ident_ids_by_module: Arc<Mutex<MutMap<ModuleId, IdentIds>>>,
header: &PlatformHeader<'a>, header: &PlatformHeader<'a>,
module_timing: ModuleTiming, module_timing: ModuleTiming,
) -> Result<(ModuleId, Msg<'a>), LoadingProblem<'a>> { ) -> (ModuleId, Msg<'a>) {
let provides: &'a [Located<ExposesEntry<'a, &'a str>>] = let provides: &'a [Located<ExposesEntry<'a, &'a str>>] =
header.provides.clone().into_bump_slice(); header.provides.clone().into_bump_slice();
Ok(send_header_two( send_header_two(
arena, arena,
filename, filename,
shorthand, shorthand,
@ -3110,7 +3110,7 @@ fn fabricate_pkg_config_module<'a>(
module_ids, module_ids,
ident_ids_by_module, ident_ids_by_module,
module_timing, module_timing,
)) )
} }
#[allow(clippy::too_many_arguments)] #[allow(clippy::too_many_arguments)]
@ -3122,7 +3122,7 @@ fn fabricate_effects_module<'a>(
mode: Mode, mode: Mode,
header: PlatformHeader<'a>, header: PlatformHeader<'a>,
module_timing: ModuleTiming, module_timing: ModuleTiming,
) -> Result<(ModuleId, Msg<'a>), LoadingProblem<'a>> { ) -> (ModuleId, Msg<'a>) {
let num_exposes = header.provides.len() + 1; let num_exposes = header.provides.len() + 1;
let mut exposed: Vec<Symbol> = Vec::with_capacity(num_exposes); let mut exposed: Vec<Symbol> = Vec::with_capacity(num_exposes);
@ -3350,7 +3350,7 @@ fn fabricate_effects_module<'a>(
module_timing, module_timing,
}; };
Ok(( (
module_id, module_id,
Msg::MadeEffectModule { Msg::MadeEffectModule {
type_shortname: effects.effect_shortname, type_shortname: effects.effect_shortname,
@ -3358,7 +3358,7 @@ fn fabricate_effects_module<'a>(
canonicalization_problems: module_output.problems, canonicalization_problems: module_output.problems,
module_docs, module_docs,
}, },
)) )
} }
fn unpack_exposes_entries<'a>( fn unpack_exposes_entries<'a>(
@ -3393,6 +3393,7 @@ fn unpack_exposes_entries<'a>(
} }
#[allow(clippy::too_many_arguments)] #[allow(clippy::too_many_arguments)]
#[allow(clippy::unnecessary_wraps)]
fn canonicalize_and_constrain<'a, F>( fn canonicalize_and_constrain<'a, F>(
arena: &'a Bump, arena: &'a Bump,
module_ids: &ModuleIds, module_ids: &ModuleIds,