diff options
author | Andreas Gampe <agampe@google.com> | 2014-11-12 08:03:48 -0800 |
---|---|---|
committer | Andreas Gampe <agampe@google.com> | 2014-11-13 15:20:37 -0800 |
commit | 0f0b4919667f418b249c497f5ad3e83fdf4437e5 (patch) | |
tree | 3a583c918c72b7d8d48b784505502f7b3fdf05e9 /core/jni/android_app_backup_FullBackup.cpp | |
parent | 537888be7d5cfa6bfa9ddd459a622ea208339735 (diff) | |
download | frameworks_base-0f0b4919667f418b249c497f5ad3e83fdf4437e5.zip frameworks_base-0f0b4919667f418b249c497f5ad3e83fdf4437e5.tar.gz frameworks_base-0f0b4919667f418b249c497f5ad3e83fdf4437e5.tar.bz2 |
Frameworks/base: Wall Werror in core/jni
Turn on -Wall -Werror in core/jni. Fix warnings.
Clang TODO: For GCC we need to turn off Wunused-but-set-variable in
the GL bindings. However, Clang doesn't have that warning and thus
complains about an unknown pragma. It is necessary to make the
pragma #ifdef-ed on the compiler being GCC.
Change-Id: I14cab48d45c2771eef0432082356c47ed44a3d7f
Diffstat (limited to 'core/jni/android_app_backup_FullBackup.cpp')
-rw-r--r-- | core/jni/android_app_backup_FullBackup.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/core/jni/android_app_backup_FullBackup.cpp b/core/jni/android_app_backup_FullBackup.cpp index 3cfaa82..5e334c9 100644 --- a/core/jni/android_app_backup_FullBackup.cpp +++ b/core/jni/android_app_backup_FullBackup.cpp @@ -73,8 +73,6 @@ static struct { static jint backupToTar(JNIEnv* env, jobject clazz, jstring packageNameObj, jstring domainObj, jstring linkdomain, jstring rootpathObj, jstring pathObj, jobject dataOutputObj) { - int ret; - // Extract the various strings, allowing for null object pointers const char* packagenamechars = (packageNameObj) ? env->GetStringUTFChars(packageNameObj, NULL) : NULL; const char* rootchars = (rootpathObj) ? env->GetStringUTFChars(rootpathObj, NULL) : NULL; |