Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit ea483350 authored by Zhi Dou's avatar Zhi Dou
Browse files

aconfig: update error message in modify_parsed_flags_based_on_mode 

This commit updates the error message in
modify_parsed_flags_based_on_mode, when there is no certain mode flags
in the corresponding library, and this change remove "." at the end of
the error message to conform to existing standard in this project.

Bug: 311152507
Test: atest aconfig.test
Change-Id: Ib120e7f7252981bbeb6ca1cf785da24f329507b4
parent 34785f42

tools/aconfig/src/commands.rs

+2 −5
Original line number Diff line number Diff line
@@ -339,7 +339,7 @@ pub fn modify_parsed_flags_based_on_mode( 
        }

    };

    if modified_parsed_flags.is_empty() {

        bail!("{} library contains no exported flags.", codegen_mode);

        bail!("{codegen_mode} library contains no {codegen_mode} flags");

    }



    Ok(modified_parsed_flags)
@@ -653,9 +653,6 @@ mod tests { 
        parsed_flags.parsed_flag.retain_mut(|pf| !pf.is_exported());

        let error =

            modify_parsed_flags_based_on_mode(parsed_flags, CodegenMode::Exported).unwrap_err();

        assert_eq!(

            format!("{} library contains no exported flags.", CodegenMode::Exported),

            format!("{:?}", error)

        );

        assert_eq!("exported library contains no exported flags", format!("{:?}", error));

    }

}