diff options
| author | Daniel Malea <daniel.malea@intel.com> | 2012-12-07 22:21:08 +0000 |
|---|---|---|
| committer | Daniel Malea <daniel.malea@intel.com> | 2012-12-07 22:21:08 +0000 |
| commit | a85e6b6c323d97c3c5b3c2b45f15ef3066aa9e79 (patch) | |
| tree | 31449c19dec1fb8ed688c82c056fc1dc880ba6ae /lldb/source/Core | |
| parent | 84b2a795705d994d2a333d49a00ff7132627cbe3 (diff) | |
| download | bcm5719-llvm-a85e6b6c323d97c3c5b3c2b45f15ef3066aa9e79.tar.gz bcm5719-llvm-a85e6b6c323d97c3c5b3c2b45f15ef3066aa9e79.zip | |
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
Diffstat (limited to 'lldb/source/Core')
| -rw-r--r-- | lldb/source/Core/CXXFormatterFunctions.cpp | 2 | ||||
| -rw-r--r-- | lldb/source/Core/Log.cpp | 2 | ||||
| -rw-r--r-- | lldb/source/Core/ValueObject.cpp | 4 |
3 files changed, 4 insertions, 4 deletions
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; |

