summaryrefslogtreecommitdiffstats
path: root/core/jni/android_app_backup_FullBackup.cpp
diff options
context:
space:
mode:
authorAndreas Gampe <agampe@google.com>2014-11-12 08:03:48 -0800
committerAndreas Gampe <agampe@google.com>2014-11-13 15:20:37 -0800
commit0f0b4919667f418b249c497f5ad3e83fdf4437e5 (patch)
tree3a583c918c72b7d8d48b784505502f7b3fdf05e9 /core/jni/android_app_backup_FullBackup.cpp
parent537888be7d5cfa6bfa9ddd459a622ea208339735 (diff)
downloadframeworks_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.cpp2
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;