aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorivan.penkov@gmail.com <ivan.penkov@gmail.com@4c0a9323-5329-0410-9bdc-e9ce6186880e>2014-02-28 10:21:20 +0000
committerivan.penkov@gmail.com <ivan.penkov@gmail.com@4c0a9323-5329-0410-9bdc-e9ce6186880e>2014-02-28 10:21:20 +0000
commit1df13fed15324d3f9e815b84588a758f67834d7f (patch)
treeb3f3f7973478fb904f31c240032d8e44b8240739
parentRenaming file COPYING to LICENSE and appending the disclaimer from src/common... (diff)
downloadbreakpad-1df13fed15324d3f9e815b84588a758f67834d7f.tar.xz
Fixing compiler warnings:
- Building Breakpad in Xcode with arm64 architecture. - iOS Patches provided by: Ian Hickson and Greg Vance. R=mark@chromium.org Review URL: https://breakpad.appspot.com/1184003 git-svn-id: http://google-breakpad.googlecode.com/svn/trunk@1286 4c0a9323-5329-0410-9bdc-e9ce6186880e
-rw-r--r--src/client/ios/Breakpad.mm4
-rw-r--r--src/client/mac/handler/minidump_generator.cc6
-rw-r--r--src/common/mac/file_id.cc2
-rw-r--r--src/common/mac/macho_id.cc2
-rw-r--r--src/common/memory.h4
5 files changed, 9 insertions, 9 deletions
diff --git a/src/client/ios/Breakpad.mm b/src/client/ios/Breakpad.mm
index 65bf51e9..66125597 100644
--- a/src/client/ios/Breakpad.mm
+++ b/src/client/ios/Breakpad.mm
@@ -98,7 +98,7 @@ class ProtectedMemoryLocker {
: mutex_(mutex),
allocator_(allocator) {
// Lock the mutex
- int rv = pthread_mutex_lock(mutex_);
+ __attribute__((unused)) int rv = pthread_mutex_lock(mutex_);
assert(rv == 0);
// Unprotect the memory
@@ -110,7 +110,7 @@ class ProtectedMemoryLocker {
allocator_->Protect();
// Then unlock the mutex
- int rv = pthread_mutex_unlock(mutex_);
+ __attribute__((unused)) int rv = pthread_mutex_unlock(mutex_);
assert(rv == 0);
};
diff --git a/src/client/mac/handler/minidump_generator.cc b/src/client/mac/handler/minidump_generator.cc
index 7b166e58..38baa04e 100644
--- a/src/client/mac/handler/minidump_generator.cc
+++ b/src/client/mac/handler/minidump_generator.cc
@@ -1479,13 +1479,13 @@ bool MinidumpGenerator::WriteModuleListStream(
module_list_stream->stream_type = MD_MODULE_LIST_STREAM;
module_list_stream->location = list.location();
- list.get()->number_of_modules = image_count;
+ list.get()->number_of_modules = static_cast<uint32_t>(image_count);
// Write out the executable module as the first one
MDRawModule module;
uint32_t executableIndex = FindExecutableModule();
- if (!WriteModuleStream(executableIndex, &module)) {
+ if (!WriteModuleStream(static_cast<unsigned>(executableIndex), &module)) {
return false;
}
@@ -1494,7 +1494,7 @@ bool MinidumpGenerator::WriteModuleListStream(
for (uint32_t i = 0; i < image_count; ++i) {
if (i != executableIndex) {
- if (!WriteModuleStream(i, &module)) {
+ if (!WriteModuleStream(static_cast<unsigned>(i), &module)) {
return false;
}
diff --git a/src/common/mac/file_id.cc b/src/common/mac/file_id.cc
index a2ee320b..c5d7ed56 100644
--- a/src/common/mac/file_id.cc
+++ b/src/common/mac/file_id.cc
@@ -61,7 +61,7 @@ bool FileID::FileIdentifier(unsigned char identifier[16]) {
unsigned char buffer[4096 * 2];
size_t buffer_size = sizeof(buffer);
while ((buffer_size = read(fd, buffer, buffer_size) > 0)) {
- MD5Update(&md5, buffer, buffer_size);
+ MD5Update(&md5, buffer, static_cast<unsigned>(buffer_size));
}
close(fd);
diff --git a/src/common/mac/macho_id.cc b/src/common/mac/macho_id.cc
index aa2cfc83..09a2a82e 100644
--- a/src/common/mac/macho_id.cc
+++ b/src/common/mac/macho_id.cc
@@ -125,7 +125,7 @@ void MachoID::UpdateCRC(unsigned char *bytes, size_t size) {
}
void MachoID::UpdateMD5(unsigned char *bytes, size_t size) {
- MD5Update(&md5_context_, bytes, size);
+ MD5Update(&md5_context_, bytes, static_cast<unsigned>(size));
}
void MachoID::Update(MachoWalker *walker, off_t offset, size_t size) {
diff --git a/src/common/memory.h b/src/common/memory.h
index f2e39104..a4636059 100644
--- a/src/common/memory.h
+++ b/src/common/memory.h
@@ -142,10 +142,10 @@ class PageAllocator {
size_t num_pages; // the number of pages in this set.
};
- const unsigned page_size_;
+ const size_t page_size_;
PageHeader *last_;
uint8_t *current_page_;
- unsigned page_offset_;
+ size_t page_offset_;
};
// Wrapper to use with STL containers