Refactor RunInLocale a bit

Change-Id: If60877b29bb1263161ec85c99320ad07b7dd4245
main
Tadashi G. Takaoka 2014-04-17 16:59:19 +09:00
parent b21edc4402
commit 5c490e5da5
1 changed files with 10 additions and 12 deletions

View File

@ -30,25 +30,23 @@ public abstract class RunInLocale<T> {
* Execute {@link #job(Resources)} method in specified system locale exclusively.
*
* @param res the resources to use.
* @param newLocale the locale to change to.
* @param newLocale the locale to change to. Run in system locale if null.
* @return the value returned from {@link #job(Resources)}.
*/
public T runInLocale(final Resources res, final Locale newLocale) {
synchronized (sLockForRunInLocale) {
final Configuration conf = res.getConfiguration();
final Locale oldLocale = conf.locale;
final boolean needsChange = (newLocale != null && !newLocale.equals(oldLocale));
try {
if (needsChange) {
conf.locale = newLocale;
res.updateConfiguration(conf, null);
final Configuration savedConf = res.getConfiguration();
if (newLocale == null || newLocale.equals(savedConf.locale)) {
return job(res);
}
final Configuration newConf = new Configuration();
newConf.setTo(savedConf);
newConf.setLocale(newLocale);
try {
res.updateConfiguration(newConf, null);
return job(res);
} finally {
if (needsChange) {
conf.locale = oldLocale;
res.updateConfiguration(conf, null);
}
res.updateConfiguration(savedConf, null);
}
}
}