Fix build errors encountered by some compilers (#4114)

b/365029210
youtube · Sep 17, 2024 · 6438fd0 · 6438fd0
1 parent 514e328
commit 6438fd0
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 2 deletions.
diff --git a/starboard/shared/pthread/thread_priority.cc b/starboard/shared/pthread/thread_priority.cc
@@ -35,6 +35,7 @@ int SbPriorityToNice(SbThreadPriority priority) {
     case kSbThreadPriorityRealTime:
       return -19;
   }
+  return 0;
 }
 
 SbThreadPriority NiceToSbPriority(int nice) {
@@ -56,6 +57,7 @@ SbThreadPriority NiceToSbPriority(int nice) {
   if (nice == -19) {
     return kSbThreadPriorityRealTime;
   }
+  return kSbThreadPriorityNormal;
 }
 
 }  // namespace

diff --git a/third_party/crashpad/crashpad/client/crash_report_database_generic.cc b/third_party/crashpad/crashpad/client/crash_report_database_generic.cc
@@ -662,7 +662,7 @@ OperationStatus CrashReportDatabaseGeneric::RemoveOldReports(
       all_reports.end(), completed_reports.begin(), completed_reports.end());
   std::sort(all_reports.begin(), all_reports.end(), WasCreatedSooner);
 
-  while (all_reports.size() > num_reports_to_keep) {
+  while (all_reports.size() > static_cast<unsigned int>(num_reports_to_keep)) {
     OperationStatus os = DeleteReport((*all_reports.begin()).uuid);
     if (os != kNoError) {
       return os;

diff --git a/third_party/crashpad/crashpad/snapshot/linux/process_snapshot_linux.cc b/third_party/crashpad/crashpad/snapshot/linux/process_snapshot_linux.cc
@@ -393,7 +393,7 @@ void ProcessSnapshotLinux::InitializeModules(
   }
 
   std::vector<uint8_t> build_id(evergreen_info.build_id_length);
-  for (int i = 0; i < build_id.size(); i++) {
+  for (unsigned int i = 0; i < build_id.size(); i++) {
     build_id[i] = reinterpret_cast<uint8_t*>(evergreen_info.build_id)[i];
   }