diff options
Diffstat (limited to 'cmds/screencap')
-rw-r--r-- | cmds/screencap/Android.mk | 12 | ||||
-rw-r--r-- | cmds/screencap/screencap.cpp | 14 |
2 files changed, 17 insertions, 9 deletions
diff --git a/cmds/screencap/Android.mk b/cmds/screencap/Android.mk index 5c11b75..b0dc422 100644 --- a/cmds/screencap/Android.mk +++ b/cmds/screencap/Android.mk @@ -2,13 +2,13 @@ LOCAL_PATH:= $(call my-dir) include $(CLEAR_VARS) LOCAL_SRC_FILES:= \ - screencap.cpp + screencap.cpp LOCAL_SHARED_LIBRARIES := \ - libcutils \ - libutils \ - libbinder \ - libskia \ + libcutils \ + libutils \ + libbinder \ + libskia \ libui \ libgui @@ -16,4 +16,6 @@ LOCAL_MODULE:= screencap LOCAL_MODULE_TAGS := optional +LOCAL_CFLAGS += -Wall -Werror -Wunused -Wunreachable-code + include $(BUILD_EXECUTABLE) diff --git a/cmds/screencap/screencap.cpp b/cmds/screencap/screencap.cpp index 1ddbecb..b0aee7b 100644 --- a/cmds/screencap/screencap.cpp +++ b/cmds/screencap/screencap.cpp @@ -32,10 +32,14 @@ #include <ui/PixelFormat.h> +// TODO: Fix Skia. +#pragma GCC diagnostic push +#pragma GCC diagnostic ignored "-Wunused-parameter" #include <SkImageEncoder.h> #include <SkBitmap.h> #include <SkData.h> #include <SkStream.h> +#pragma GCC diagnostic pop using namespace android; @@ -129,7 +133,7 @@ int main(int argc, char** argv) argv += optind; int fd = -1; - const char* fn; + const char* fn = NULL; if (argc == 0) { fd = dup(STDOUT_FILENO); } else if (argc == 1) { @@ -153,7 +157,7 @@ int main(int argc, char** argv) void const* mapbase = MAP_FAILED; ssize_t mapsize = -1; - void const* base = 0; + void const* base = NULL; uint32_t w, s, h, f; size_t size = 0; @@ -190,7 +194,7 @@ int main(int argc, char** argv) } } - if (base) { + if (base != NULL) { if (png) { const SkImageInfo info = SkImageInfo::Make(w, h, flinger2skia(f), kPremul_SkAlphaType); @@ -202,7 +206,9 @@ int main(int argc, char** argv) SkData* streamData = stream.copyToData(); write(fd, streamData->data(), streamData->size()); streamData->unref(); - notifyMediaScanner(fn); + if (fn != NULL) { + notifyMediaScanner(fn); + } } else { write(fd, &w, 4); write(fd, &h, 4); |