From a85e6b6c323d97c3c5b3c2b45f15ef3066aa9e79 Mon Sep 17 00:00:00 2001 From: Daniel Malea Date: Fri, 7 Dec 2012 22:21:08 +0000 Subject: Fix a few more clang (3.2) warnings on Linux: - remove unused members - add NO_PEDANTIC to selected Makefiles - fix return values (removed NULL as needed) - disable warning about four-char-constants - remove unneeded const from operator*() declaration - add missing lambda function return types - fix printf() with no format string - change sizeof to use a type name instead of variable name - fix Linux ProcessMonitor.cpp to be 32/64 bit friendly - disable warnings emitted by swig-generated C++ code Patch by Matt Kopec! llvm-svn: 169645 --- lldb/source/Core/CXXFormatterFunctions.cpp | 2 +- lldb/source/Core/Log.cpp | 2 +- lldb/source/Core/ValueObject.cpp | 4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) (limited to 'lldb/source/Core') diff --git a/lldb/source/Core/CXXFormatterFunctions.cpp b/lldb/source/Core/CXXFormatterFunctions.cpp index 6f728afac8a..f8f8fa57cb8 100644 --- a/lldb/source/Core/CXXFormatterFunctions.cpp +++ b/lldb/source/Core/CXXFormatterFunctions.cpp @@ -40,7 +40,7 @@ lldb_private::formatters::ExtractValueFromObjCExpression (ValueObject &valobj, if (!selector || !*selector) return false; StreamString expr; - expr.Printf("(%s)[(id)0x%llx %s]",target_type,valobj.GetPointerValue(),selector); + expr.Printf("(%s)[(id)0x%" PRIx64 " %s]",target_type,valobj.GetPointerValue(),selector); ExecutionContext exe_ctx (valobj.GetExecutionContextRef()); lldb::ValueObjectSP result_sp; Target* target = exe_ctx.GetTargetPtr(); diff --git a/lldb/source/Core/Log.cpp b/lldb/source/Core/Log.cpp index 14e8ff82467..f8e422e2df7 100644 --- a/lldb/source/Core/Log.cpp +++ b/lldb/source/Core/Log.cpp @@ -102,7 +102,7 @@ Log::PrintfWithFlagsVarArg (uint32_t flags, const char *format, va_list args) if (m_options.Test (LLDB_LOG_OPTION_PREPEND_TIMESTAMP)) { struct timeval tv = TimeValue::Now().GetAsTimeVal(); - header.Printf ("%9ld.%6.6" PRIi32 " ", tv.tv_sec, tv.tv_usec); + header.Printf ("%9ld.%6.6d ", tv.tv_sec, (int32_t)tv.tv_usec); } // Add the process and thread if requested diff --git a/lldb/source/Core/ValueObject.cpp b/lldb/source/Core/ValueObject.cpp index 8e4495396c0..c95095f7c02 100644 --- a/lldb/source/Core/ValueObject.cpp +++ b/lldb/source/Core/ValueObject.cpp @@ -2527,7 +2527,7 @@ ValueObject::GetValueForExpressionPath_Impl(const char* expression_cstr, if (child_valobj_sp.get()) // we know we are done, so just return { - *first_unparsed = '\0'; + *first_unparsed = ""; *reason_to_stop = ValueObject::eExpressionPathScanEndReasonEndOfString; *final_result = ValueObject::eExpressionPathEndResultTypePlain; return child_valobj_sp; @@ -2551,7 +2551,7 @@ ValueObject::GetValueForExpressionPath_Impl(const char* expression_cstr, // so we hit the "else" branch, and return an error if(child_valobj_sp.get()) // if it worked, just return { - *first_unparsed = '\0'; + *first_unparsed = ""; *reason_to_stop = ValueObject::eExpressionPathScanEndReasonEndOfString; *final_result = ValueObject::eExpressionPathEndResultTypePlain; return child_valobj_sp; -- cgit v1.2.3