Review fixes.

This commit is contained in:
Dmitry Yunitsky 2016-03-22 11:44:14 +03:00 committed by Sergey Yershov
parent f18e275128
commit 8c49302db7
2 changed files with 11 additions and 9 deletions

View file

@ -292,9 +292,10 @@ JNIEXPORT jobjectArray JNICALL
Java_com_mapswithme_maps_editor_Editor_nativeGetCuisines(JNIEnv * env, jclass clazz)
{
osm::TAllCuisines const & cuisines = osm::Cuisines::Instance().AllSupportedCuisines();
vector<string> keys(cuisines.size());
for (size_t i = 0; i < cuisines.size(); i++)
keys[i] = cuisines[i].first;
vector<string> keys;
keys.reserve(cuisines.size());
for (TCuisine const & cuisine : cuisines)
keys.push_back(cuisine.first);
return ToJavaArray(env, keys);
}
@ -308,9 +309,10 @@ JNIEXPORT jobjectArray JNICALL
Java_com_mapswithme_maps_editor_Editor_nativeGetCuisinesTranslations(JNIEnv * env, jclass clazz)
{
osm::TAllCuisines const & cuisines = osm::Cuisines::Instance().AllSupportedCuisines();
vector<string> keys(cuisines.size());
for (size_t i = 0; i < cuisines.size(); i++)
keys[i] = cuisines[i].second;
vector<string> keys;
keys.reserve(cuisines.size());
for (TCuisine const & cuisine : cuisines)
keys.push_back(cuisine.second);
return ToJavaArray(env, keys);
}

View file

@ -114,7 +114,7 @@
android:layout_alignLeft="@id/street"
android:layout_alignStart="@id/street"
android:layout_below="@id/street"
android:layout_marginTop="6dp"
android:layout_marginTop="@dimen/margin_quarter_plus"
android:background="@color/divider"/>
</RelativeLayout>
@ -253,7 +253,7 @@
android:layout_height="wrap_content"
android:layout_marginLeft="@dimen/editor_margin_left"
android:layout_marginStart="@dimen/editor_margin_left"
android:text="@string/street"
android:text="@string/cuisine"
android:textAppearance="@style/MwmTextAppearance.Body4"/>
<TextView
@ -273,7 +273,7 @@
android:layout_alignLeft="@id/cuisine"
android:layout_alignStart="@id/cuisine"
android:layout_below="@id/cuisine"
android:layout_marginTop="6dp"
android:layout_marginTop="@dimen/margin_quarter_plus"
android:background="@color/divider"/>
</RelativeLayout>