[Android] PreferencesActivity: clean up the cleanup
Jean-Baptiste Kempf
jb at videolan.org
Wed Mar 13 15:50:38 CET 2013
On 13 Mar, Edward Wang wrote :
> vlc-ports/android | branch: master | Edward Wang <edward.c.wang at compdigitec.com> | Wed Mar 13 10:40:56 2013 -0400| [2f9d6de538a871b1bbe6ea5aee9af6de311a60ca] | committer: Edward Wang
>
> PreferencesActivity: clean up the cleanup
This is wrong.
First, the .java part should not be in this commit.
This is a blatant lie:
+ <string name="chroma_format_summary">Android 2.3 and later
only.\nRGB 32-bit: default chroma\nRGB 16-bit: better performance but
lower quality\nYUV: best performance but does not work on all
devices</string>
RGB16 works fine on 2.1. The previous placement was correct.
+ <string name="enable_time_stretching_audio_summary">Speed up and
slow down audio without changing the pitch (requires a fast
device).</string>
This is wrong, people have no idea what pitch is.
+ <string name="subtitles_text_encoding">Subtitle text
encoding</string>
There are more than one subtitle.
Moving android:entries="@array/chroma_formats" is wrong too. There is a
good reason to put it at the top of the perf category...
Reverting the moving clear_media_db to other needs to be discuss. You
don't just revert things without explaning...
Removing android:summary="@string/screen_orientation_summary" needs also
a good reason.
Best regards,
--
Jean-Baptiste Kempf
http://www.jbkempf.com/ - +33 672 704 734
Sent from my Electronic Device
More information about the Android
mailing list