Subsurface 4.5 (Win10) crash after adding dive with planner

Dirk Hohndel dirk at hohndel.org
Sat Oct 17 22:00:45 PDT 2015


> On Oct 17, 2015, at 9:58 PM, Miika Turkia <miika.turkia at gmail.com> wrote:
> 
> On Sun, Oct 18, 2015 at 7:51 AM, Dirk Hohndel <dirk at hohndel.org> wrote:
>> 
>>> On Oct 17, 2015, at 7:39 PM, Linus Torvalds <torvalds at linux-foundation.org> wrote:
>>> 
>>> On Sat, Oct 17, 2015 at 7:32 PM, Linus Torvalds
>>> <torvalds at linux-foundation.org> wrote:
>>>> 
>>>> But right you are. That explains it. The German translation format
>>>> string is corrupt.
>>> 
>>> True for both Swiss and German locale (de_CH and de_DE).
>> 
>> Yep, thanks Lubomir and Linus, that's clearly it.
>> I just got back from dinner and looked at it on the phone while there and something looked
>> odd about the format string but I just figured "it's the phone screen"...
>> 
>> I'll fix that on Transifex. And I'll check the other languages as well, Benjamin.
> 
> We are slowly getting a nice list of formatting options that must be
> rejected before release. Time for some scripting, I assume.
> Unfortunately all % signs are not marking an argument but might be
> literal % in the output...

It should be a valid requirement to say that the number of '%' in the original
string and the translation must be the same.

That would have been enough to catch this one.

I'm done reviewing. Only wrong in the two German translations. I also realized
that we had a near identical string that was translated in many of the languages
but that this one was missing in some of them, so I did some "pattern matching
translations".

Benjamin: the string is untranslated in Hebrew.

/D


More information about the subsurface mailing list