From ef021b1e8a00f20dcfcd4aea44f8c0795398284b Mon Sep 17 00:00:00 2001
From: Sylvain Berfini <sylvain.berfini@belledonne-communications.com>
Date: Fri, 30 Nov 2012 10:32:11 +0100
Subject: [PATCH] Add H264 log warning if not available

---
 res/layout/history_cell_simple.xml          | 2 +-
 src/org/linphone/HistorySimpleFragment.java | 3 +--
 src/org/linphone/LinphoneManager.java       | 7 ++++++-
 src/org/linphone/PreferencesFragment.java   | 1 +
 4 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/res/layout/history_cell_simple.xml b/res/layout/history_cell_simple.xml
index 76d7bf0b8..57bf2af0c 100644
--- a/res/layout/history_cell_simple.xml
+++ b/res/layout/history_cell_simple.xml
@@ -16,7 +16,7 @@
 	    android:src="@drawable/call_status_incoming"
 	    android:layout_marginLeft="5dp"/>
 	    
-	<TextView 
+	<TextView
 	    android:id="@+id/sipUri"
 	    android:lines="1"
 	    android:textSize="30sp"
diff --git a/src/org/linphone/HistorySimpleFragment.java b/src/org/linphone/HistorySimpleFragment.java
index 3a1614196..daae4a6a0 100644
--- a/src/org/linphone/HistorySimpleFragment.java
+++ b/src/org/linphone/HistorySimpleFragment.java
@@ -362,7 +362,6 @@ public class HistorySimpleFragment extends Fragment implements OnClickListener,
 			}
 
 			return view;
-		}
-		  
+		}		  
 	  }
 }
\ No newline at end of file
diff --git a/src/org/linphone/LinphoneManager.java b/src/org/linphone/LinphoneManager.java
index 5fcdd3c38..7f84bc631 100644
--- a/src/org/linphone/LinphoneManager.java
+++ b/src/org/linphone/LinphoneManager.java
@@ -845,22 +845,27 @@ public final class LinphoneManager implements LinphoneCoreListener {
 	private void enableDisableVideoCodecs(PayloadType videoCodec) throws LinphoneCoreException {
 		String mime = videoCodec.getMime();
 		int key;
+		int defaultValueKey;
 		
 		if ("MP4V-ES".equals(mime)) {
 			key = R.string.pref_video_codec_mpeg4_key;
+			defaultValueKey = R.bool.pref_video_codec_mpeg4_default;
 		} else if ("H264".equals(mime)) {
 			key = R.string.pref_video_codec_h264_key;
+			defaultValueKey = R.bool.pref_video_codec_h264_default;
 		} else if ("H263-1998".equals(mime)) {
 			key = R.string.pref_video_codec_h263_key;
+			defaultValueKey = R.bool.pref_video_codec_h263_default;
 		} else if ("VP8".equals(mime)) {
 			key = R.string.pref_video_codec_vp8_key;
+			defaultValueKey = R.bool.pref_video_codec_vp8_default;
 		} else {
 			Log.e("Unhandled video codec ", mime);
 			mLc.enablePayloadType(videoCodec, false);
 			return;
 		}
 
-		boolean enable= getPrefBoolean(key,false);
+		boolean enable = getPrefBoolean(key, mR.getBoolean(defaultValueKey));
 		mLc.enablePayloadType(videoCodec, enable);
 	}
 
diff --git a/src/org/linphone/PreferencesFragment.java b/src/org/linphone/PreferencesFragment.java
index d857754b8..1fc95be91 100644
--- a/src/org/linphone/PreferencesFragment.java
+++ b/src/org/linphone/PreferencesFragment.java
@@ -167,6 +167,7 @@ public class PreferencesFragment extends PreferencesListFragment implements EcCa
 		if (!Version.hasNeon())
 		{
 			// Android without neon doesn't support H264
+			Log.w("No NEON available, disabling H264");
 			findPreference(R.string.pref_video_codec_h264_key).setEnabled(false);
 			findPreference(R.string.pref_video_codec_h264_key).setDefaultValue(false);
 		}