Hahaha it was me \o/

Seriously, I don't care about localization in test.
A long time ago we had a discussion about localization in tests.
I would have preferred to have some tests about localization mechanism and no localization in the others.
The goal of all tests is *not* to check localization.
But the final decision was different.

In your commit you have mix localization and no localization, so I update to full localization that's all.

Antoine
Le 14/06/2019 à 00:38, Samuel Gougeon a écrit :
Hello Clément,

Le 13/06/2019 à 15:02, Clément David a écrit :
Hello Samuel,

There should be no real difference, having ENGLISH IMPOSED might have subtle bugs related to the system locales whereas using gettext() will rely on an extra Scilab function call (so will be less a
"unit test").

IMHO commits having only this kind of diff is useless; there should be something else, could you
point us the related code-review ?

Both cases i had in mind were not commited just for this. But this change was made in addition before merging.
Here is one case :

https://codereview.scilab.org/#/c/20723/6..7

There was another one around the 6.0.2 release, but i failed refinding it.

I agree about the extra function call. But as i wrote before, i got aware that using gettext() rather than ENGLISH IMPOSED is more robust for (rare but possible) cases having a translation even for en_US.

Regards
Samuel

_______________________________________________
dev mailing list
dev@lists.scilab.org
http://lists.scilab.org/mailman/listinfo/dev

_______________________________________________
dev mailing list
dev@lists.scilab.org
http://lists.scilab.org/mailman/listinfo/dev

Reply via email to