From 5f112cb1743aa832bcb14530bd98cbacf0f49df4 Mon Sep 17 00:00:00 2001 From: Leonard Mosescu Date: Thu, 6 Jul 2017 10:58:40 -0700 Subject: A couple of minor fixes 1. Fixing ExceptionHandlerTest.FirstChanceHandlerRuns: exit() is not an async-signal-safe function (http://man7.org/linux/man-pages/man7/signal-safety.7.html) 2. Fixing entry point signature in minidump_dump Changed "const char* argv[]" to "char* argv[]" to match the standard entry point signature 3. Updating .gitignore to exclude unit test artifacts Change-Id: I9662898d0bd97769621fb6476a720105821c60f0 Reviewed-on: https://chromium-review.googlesource.com/562356 Reviewed-by: Mark Mentovai Reviewed-by: Ivan Penkov Reviewed-by: Joshua Peraza --- src/processor/minidump_dump.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'src/processor') diff --git a/src/processor/minidump_dump.cc b/src/processor/minidump_dump.cc index 4d916276..b9b7555e 100644 --- a/src/processor/minidump_dump.cc +++ b/src/processor/minidump_dump.cc @@ -212,7 +212,7 @@ static bool PrintMinidumpDump(const Options& options) { //============================================================================= static void -Usage(int argc, const char *argv[], bool error) { +Usage(int argc, char *argv[], bool error) { FILE *fp = error ? stderr : stdout; fprintf(fp, @@ -228,7 +228,7 @@ Usage(int argc, const char *argv[], bool error) { //============================================================================= static void -SetupOptions(int argc, const char *argv[], Options *options) { +SetupOptions(int argc, char *argv[], Options *options) { int ch; while ((ch = getopt(argc, (char * const *)argv, "xh")) != -1) { @@ -257,7 +257,7 @@ SetupOptions(int argc, const char *argv[], Options *options) { } // namespace -int main(int argc, const char *argv[]) { +int main(int argc, char *argv[]) { Options options; BPLOG_INIT(&argc, &argv); SetupOptions(argc, argv, &options); -- cgit v1.2.1