forked from organicmaps/organicmaps
[android] Review fixes
This commit is contained in:
parent
de60c53326
commit
424418dd7e
3 changed files with 10 additions and 5 deletions
|
@ -198,8 +198,7 @@ bool HandleJavaException(JNIEnv * env)
|
|||
const jthrowable e = env->ExceptionOccurred();
|
||||
env->ExceptionDescribe();
|
||||
env->ExceptionClear();
|
||||
//LOG(LERROR, (ToNativeString(env, e)));
|
||||
LOG(LWARNING, (ToNativeString(env, e)));
|
||||
LOG(LERROR, (ToNativeString(env, e)));
|
||||
return true;
|
||||
}
|
||||
return false;
|
||||
|
|
|
@ -554,7 +554,11 @@ uber::RideRequestLinks Framework::GetUberLinks(string const & productId, ms::Lat
|
|||
return uber::Api::GetRideRequestLinks(productId, from, to);
|
||||
}
|
||||
|
||||
int Framework::ToDoAfterUpdate() const { return (int)m_work.ToDoAfterUpdate(); }
|
||||
int Framework::ToDoAfterUpdate() const
|
||||
{
|
||||
return (int) m_work.ToDoAfterUpdate();
|
||||
}
|
||||
|
||||
} // namespace android
|
||||
|
||||
//============ GLUE CODE for com.mapswithme.maps.Framework class =============//
|
||||
|
@ -755,9 +759,10 @@ JNIEXPORT jobjectArray JNICALL
|
|||
Java_com_mapswithme_maps_Framework_nativeGetOutdatedCountries(JNIEnv * env, jclass)
|
||||
{
|
||||
vector<Country const *> countries;
|
||||
vector<string> ids;
|
||||
class Storage const & storage = g_framework->GetStorage();
|
||||
Storage const & storage = g_framework->GetStorage();
|
||||
storage.GetOutdatedCountries(countries);
|
||||
|
||||
vector<string> ids;
|
||||
for (auto country : countries)
|
||||
ids.push_back(country->Name());
|
||||
|
||||
|
|
|
@ -153,6 +153,7 @@ public class Framework
|
|||
@NonNull
|
||||
public static native String[] nativeGetOutdatedCountries();
|
||||
|
||||
@UiThread
|
||||
@DoAfterUpdate
|
||||
public static native int nativeToDoAfterUpdate();
|
||||
|
||||
|
|
Loading…
Add table
Reference in a new issue