[haiku-commits] Re: r40933 - haiku/trunk/headers/os/locale

  • From: "Axel Dörfler" <axeld@xxxxxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Sun, 13 Mar 2011 22:29:24 +0100

jonas@xxxxxxxxxxx wrote:
>  #define B_TRANSLATE_APP_NAME(string) \
> -     BLocaleRoster::Default()->GetCatalog()->GetString((string), \
> -             B_TRANSLATE_APP_NAME_CONTEXT)
> +     BLocaleRoster::Default()->IsFilesystemTranslationPreferred() \
> +     ? BLocaleRoster::Default()->GetCatalog()->GetString((string), \
> +             B_TRANSLATE_APP_NAME_CONTEXT) : (string)

Since the app name should be known to the BRoster already, why not just 
add a BLocaleRoster::GetTranslatedAppName()? It wouldn't even need any 
argument AFAICT.

Bye,
   Axel.


Other related posts: