[vlc-devel] [PATCH] macosx: Attempt to localize the advanced prefs dialog
david.fuhrmann at googlemail.com
Sun Jan 29 15:21:29 CET 2012
Am 29.01.2012 um 14:54 schrieb Felix Paul Kühne:
> Hi there,
> On 29.01.2012, at 12:53, David Fuhrmann wrote:
>> Hello again,
>> I noticed that the advanced prefs dialog never translated the strings shown on the right side (even not in VLC 1.1.x). That is because of the fact that 'localizedString:' is used which does not translate anything, but only converts a c string to an NSString.
> Thanks for making me aware of that. I chose a different approach to this problem in [d285961c0b2422c8c5e7af0109263064ec8443d2]. I'll rework that code aspect for 2.0-next to have something nicer. Especially VLCMain isn't the place where string conversations should take place. This legacy code dating back to VLC 0.3.x in 2002..
Yeah, I also played around with this approach. But this leads to some problems, especially as sometimes p_item->psz_longtext (tooltip text) seems to be NULL. That leads to crashes when you select some parts in prefs as _() cannot handle NULL strings obviously.
Here, some extra ifs seems to be needed to avoid that.
More information about the vlc-devel