finchie/cli_framework - Change 73C6NOJ7W7ZCQ2THEZ4HXNZF3734VIPHI5VKQ2CCPLOA4L3JFIIQC

Fix minimal `Localize` implementation on errors

This makes the errors returned much cleaner; only the original error is shown and all other code still works normally. Before, the struct annotated with #[localize()] would be reported as missing which was unnecessary noise.

Created by finchie on March 5, 2025

73C6NOJ7W7ZCQ2THEZ4HXNZF3734VIPHI5VKQ2CCPLOA4L3JFIIQC

Dependencies

In channels

main

Change contents

Replacement in fluent_embed_derive/src/lib.rs at line 30 [4.74]

B:BD[3.907] → [2.5608:5701]

            // Generate a minimal `localize()` implementation so the error is self-contained

[3.907]

[2.5701]

            // Generate a minimal `Localize` implementation so the error is self-contained

Replacement in fluent_embed_derive/src/lib.rs at line 33 [4.74]

B:BD[2.5766] → [2.5766:5847]

                impl #ident {
                    fn localize(&self) -> String {

[2.5766]

[2.5847]

                #original_item
                impl<W: ::std::io::Write> ::fluent_embed::Localize<W> for #ident {
                    const CANONICAL_LOCALE: ::fluent_embed::icu_locale::LanguageIdentifier =
                        ::fluent_embed::icu_locale::langid!("en-US");
                    fn available_locales(&self) -> Vec<::fluent_embed::icu_locale::LanguageIdentifier> {
                        unimplemented!("Encountered error in derive macro")
                    }
        
                    fn message_for_locale(
                        &self,
                        writer: &mut W,
                        locale: &::fluent_embed::icu_locale::LanguageIdentifier,
                    ) -> Result<(), ::fluent_embed::LocalizationError> {

Insertion in fluent_embed_derive/src/lib.rs at line 56 [4.74]
[3.951]