remove warnings found with --enable-fuzzing

RESOLVED FIXED in Firefox 55

Status

()

RESOLVED FIXED
a year ago
28 days ago

People

(Reporter: rforbes, Assigned: rforbes)

Tracking

unspecified
mozilla55
Points:
---

Firefox Tracking Flags

(firefox55 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

a year ago
trying to build with --enable-fuzzing causes all kinds of warnings that make the  build fail. 



[task 2017-05-16T10:00:41.891620Z] 10:00:41     INFO -                  ^~~~~~~~~~~~~~~~~~~~~~
[task 2017-05-16T10:00:41.993199Z] 10:00:41     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:526:18: error: format specifies type 'unsigned long long' but the argument has type 'uint64_t' (aka 'unsigned long') [-Werror,-Wformat]
[task 2017-05-16T10:00:41.993279Z] 10:00:41     INFO -                   oldValue, *aValue);
[task 2017-05-16T10:00:41.993324Z] 10:00:41     INFO -                   ^~~~~~~~
[task 2017-05-16T10:00:41.993390Z] 10:00:41     INFO -  /home/worker/workspace/build/src/obj-firefox/dist/include/mozilla/ipc/Faulty.h:19:44: note: expanded from macro 'FAULTY_LOG'
[task 2017-05-16T10:00:41.993436Z] 10:00:41     INFO -      printf_stderr("[Faulty] " fmt "\n", ## args); \
[task 2017-05-16T10:00:41.993476Z] 10:00:41     INFO -                                             ^~~~
[task 2017-05-16T10:00:41.993559Z] 10:00:41     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:526:28: error: format specifies type 'unsigned long long' but the argument has type 'uint64_t' (aka 'unsigned long') [-Werror,-Wformat]
[task 2017-05-16T10:00:41.993599Z] 10:00:41     INFO -                   oldValue, *aValue);
[task 2017-05-16T10:00:41.993633Z] 10:00:41     INFO -                             ^~~~~~~
[task 2017-05-16T10:00:41.993697Z] 10:00:41     INFO -  /home/worker/workspace/build/src/obj-firefox/dist/include/mozilla/ipc/Faulty.h:19:44: note: expanded from macro 'FAULTY_LOG'
[task 2017-05-16T10:00:41.993741Z] 10:00:41     INFO -      printf_stderr("[Faulty] " fmt "\n", ## args); \
[task 2017-05-16T10:00:41.993778Z] 10:00:41     INFO -                                             ^~~~
[task 2017-05-16T10:00:41.993857Z] 10:00:41     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:545:18: error: format specifies type 'long long' but the argument has type 'int64_t' (aka 'long') [-Werror,-Wformat]
[task 2017-05-16T10:00:41.993897Z] 10:00:41     INFO -                   oldValue, *aValue);
[task 2017-05-16T10:00:41.993930Z] 10:00:41     INFO -                   ^~~~~~~~
[task 2017-05-16T10:00:41.993991Z] 10:00:41     INFO -  /home/worker/workspace/build/src/obj-firefox/dist/include/mozilla/ipc/Faulty.h:19:44: note: expanded from macro 'FAULTY_LOG'
[task 2017-05-16T10:00:41.994034Z] 10:00:41     INFO -      printf_stderr("[Faulty] " fmt "\n", ## args); \
[task 2017-05-16T10:00:41.994075Z] 10:00:41     INFO -                                             ^~~~
[task 2017-05-16T10:00:41.995645Z] 10:00:41     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:545:28: error: format specifies type 'long long' but the argument has type 'int64_t' (aka 'long') [-Werror,-Wformat]
[task 2017-05-16T10:00:41.995701Z] 10:00:41     INFO -                   oldValue, *aValue);
[task 2017-05-16T10:00:41.995738Z] 10:00:41     INFO -                             ^~~~~~~
[task 2017-05-16T10:00:41.995801Z] 10:00:41     INFO -  /home/worker/workspace/build/src/obj-firefox/dist/include/mozilla/ipc/Faulty.h:19:44: note: expanded from macro 'FAULTY_LOG'
[task 2017-05-16T10:00:41.995843Z] 10:00:41     INFO -      printf_stderr("[Faulty] " fmt "\n", ## args); \
[task 2017-05-16T10:00:41.995880Z] 10:00:41     INFO -                                             ^~~~
[task 2017-05-16T10:00:42.034767Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:109:5: error: unannotated fall-through between switch labels [-Werror,-Wimplicit-fallthrough]
[task 2017-05-16T10:00:42.034840Z] 10:00:42     INFO -      case 1:
[task 2017-05-16T10:00:42.034879Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.034961Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:344:3: note: in instantiation of function template specialization 'mozilla::ipc::FuzzIntegralType<char>' requested here
[task 2017-05-16T10:00:42.035005Z] 10:00:42     INFO -    FuzzIntegralType<char>(aValue, true);
[task 2017-05-16T10:00:42.035032Z] 10:00:42     INFO -    ^
[task 2017-05-16T10:00:42.038637Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:109:5: note: insert 'MOZ_FALLTHROUGH;' to silence this warning
[task 2017-05-16T10:00:42.038687Z] 10:00:42     INFO -      case 1:
[task 2017-05-16T10:00:42.038716Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.038743Z] 10:00:42     INFO -      MOZ_FALLTHROUGH;
[task 2017-05-16T10:00:42.038797Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:109:5: note: insert 'break;' to avoid fall-through
[task 2017-05-16T10:00:42.038822Z] 10:00:42     INFO -      case 1:
[task 2017-05-16T10:00:42.038843Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.038866Z] 10:00:42     INFO -      break;
[task 2017-05-16T10:00:42.038933Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:115:5: error: unannotated fall-through between switch labels [-Werror,-Wimplicit-fallthrough]
[task 2017-05-16T10:00:42.038957Z] 10:00:42     INFO -      case 2:
[task 2017-05-16T10:00:42.038978Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.042388Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:115:5: note: insert 'MOZ_FALLTHROUGH;' to silence this warning
[task 2017-05-16T10:00:42.042443Z] 10:00:42     INFO -      case 2:
[task 2017-05-16T10:00:42.042475Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.042507Z] 10:00:42     INFO -      MOZ_FALLTHROUGH;
[task 2017-05-16T10:00:42.042561Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:115:5: note: insert 'break;' to avoid fall-through
[task 2017-05-16T10:00:42.042586Z] 10:00:42     INFO -      case 2:
[task 2017-05-16T10:00:42.042607Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.042630Z] 10:00:42     INFO -      break;
[task 2017-05-16T10:00:42.042698Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:131:9: error: unannotated fall-through between switch labels [-Werror,-Wimplicit-fallthrough]
[task 2017-05-16T10:00:42.042725Z] 10:00:42     INFO -          case 1:
[task 2017-05-16T10:00:42.042748Z] 10:00:42     INFO -          ^
[task 2017-05-16T10:00:42.043961Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:131:9: note: insert 'MOZ_FALLTHROUGH;' to silence this warning
[task 2017-05-16T10:00:42.044307Z] 10:00:42     INFO -          case 1:
[task 2017-05-16T10:00:42.044629Z] 10:00:42     INFO -          ^
[task 2017-05-16T10:00:42.044966Z] 10:00:42     INFO -          MOZ_FALLTHROUGH;
[task 2017-05-16T10:00:42.045358Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:131:9: note: insert 'break;' to avoid fall-through
[task 2017-05-16T10:00:42.045680Z] 10:00:42     INFO -          case 1:
[task 2017-05-16T10:00:42.045995Z] 10:00:42     INFO -          ^
[task 2017-05-16T10:00:42.046314Z] 10:00:42     INFO -          break;
[task 2017-05-16T10:00:42.047678Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:122:5: error: unannotated fall-through between switch labels [-Werror,-Wimplicit-fallthrough]
[task 2017-05-16T10:00:42.047987Z] 10:00:42     INFO -      default:
[task 2017-05-16T10:00:42.048266Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.060007Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:122:5: note: insert 'MOZ_FALLTHROUGH;' to silence this warning
[task 2017-05-16T10:00:42.060079Z] 10:00:42     INFO -      default:
[task 2017-05-16T10:00:42.060108Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.060137Z] 10:00:42     INFO -      MOZ_FALLTHROUGH;
[task 2017-05-16T10:00:42.060193Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:122:5: note: insert 'break;' to avoid fall-through
[task 2017-05-16T10:00:42.060219Z] 10:00:42     INFO -      default:
[task 2017-05-16T10:00:42.060241Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.060265Z] 10:00:42     INFO -      break;
[task 2017-05-16T10:00:42.060334Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:109:5: error: unannotated fall-through between switch labels [-Werror,-Wimplicit-fallthrough]
[task 2017-05-16T10:00:42.060363Z] 10:00:42     INFO -      case 1:
[task 2017-05-16T10:00:42.060386Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.060466Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:363:3: note: in instantiation of function template specialization 'mozilla::ipc::FuzzIntegralType<unsigned char>' requested here
[task 2017-05-16T10:00:42.060506Z] 10:00:42     INFO -    FuzzIntegralType<unsigned char>(aValue, true);
[task 2017-05-16T10:00:42.060529Z] 10:00:42     INFO -    ^
[task 2017-05-16T10:00:42.070974Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:109:5: note: insert 'MOZ_FALLTHROUGH;' to silence this warning
[task 2017-05-16T10:00:42.071044Z] 10:00:42     INFO -      case 1:
[task 2017-05-16T10:00:42.071073Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.071104Z] 10:00:42     INFO -      MOZ_FALLTHROUGH;
[task 2017-05-16T10:00:42.071159Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:109:5: note: insert 'break;' to avoid fall-through
[task 2017-05-16T10:00:42.071186Z] 10:00:42     INFO -      case 1:
[task 2017-05-16T10:00:42.071208Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.071232Z] 10:00:42     INFO -      break;
[task 2017-05-16T10:00:42.071300Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:115:5: error: unannotated fall-through between switch labels [-Werror,-Wimplicit-fallthrough]
[task 2017-05-16T10:00:42.071329Z] 10:00:42     INFO -      case 2:
[task 2017-05-16T10:00:42.071351Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.077283Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:115:5: note: insert 'MOZ_FALLTHROUGH;' to silence this warning
[task 2017-05-16T10:00:42.077347Z] 10:00:42     INFO -      case 2:
[task 2017-05-16T10:00:42.077377Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.077406Z] 10:00:42     INFO -      MOZ_FALLTHROUGH;
[task 2017-05-16T10:00:42.077462Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:115:5: note: insert 'break;' to avoid fall-through
[task 2017-05-16T10:00:42.077489Z] 10:00:42     INFO -      case 2:
[task 2017-05-16T10:00:42.077511Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.077536Z] 10:00:42     INFO -      break;
[task 2017-05-16T10:00:42.077604Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:131:9: error: unannotated fall-through between switch labels [-Werror,-Wimplicit-fallthrough]
[task 2017-05-16T10:00:42.077633Z] 10:00:42     INFO -          case 1:
[task 2017-05-16T10:00:42.077657Z] 10:00:42     INFO -          ^
[task 2017-05-16T10:00:42.088269Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:131:9: note: insert 'MOZ_FALLTHROUGH;' to silence this warning
[task 2017-05-16T10:00:42.088343Z] 10:00:42     INFO -          case 1:
[task 2017-05-16T10:00:42.088379Z] 10:00:42     INFO -          ^
[task 2017-05-16T10:00:42.088413Z] 10:00:42     INFO -          MOZ_FALLTHROUGH;
[task 2017-05-16T10:00:42.088470Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:131:9: note: insert 'break;' to avoid fall-through
[task 2017-05-16T10:00:42.088512Z] 10:00:42     INFO -          case 1:
[task 2017-05-16T10:00:42.088542Z] 10:00:42     INFO -          ^
[task 2017-05-16T10:00:42.088570Z] 10:00:42     INFO -          break;
[task 2017-05-16T10:00:42.088644Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:122:5: error: unannotated fall-through between switch labels [-Werror,-Wimplicit-fallthrough]
[task 2017-05-16T10:00:42.088673Z] 10:00:42     INFO -      default:
[task 2017-05-16T10:00:42.088695Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.095652Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:122:5: note: insert 'MOZ_FALLTHROUGH;' to silence this warning
[task 2017-05-16T10:00:42.095717Z] 10:00:42     INFO -      default:
[task 2017-05-16T10:00:42.095747Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.095777Z] 10:00:42     INFO -      MOZ_FALLTHROUGH;
[task 2017-05-16T10:00:42.095832Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:122:5: note: insert 'break;' to avoid fall-through
[task 2017-05-16T10:00:42.095859Z] 10:00:42     INFO -      default:
[task 2017-05-16T10:00:42.095882Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.095906Z] 10:00:42     INFO -      break;
[task 2017-05-16T10:00:42.095974Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:109:5: error: unannotated fall-through between switch labels [-Werror,-Wimplicit-fallthrough]
[task 2017-05-16T10:00:42.096001Z] 10:00:42     INFO -      case 1:
[task 2017-05-16T10:00:42.096024Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.096102Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:382:3: note: in instantiation of function template specialization 'mozilla::ipc::FuzzIntegralType<short>' requested here
[task 2017-05-16T10:00:42.096141Z] 10:00:42     INFO -    FuzzIntegralType<int16_t>(aValue, true);
[task 2017-05-16T10:00:42.096163Z] 10:00:42     INFO -    ^
[task 2017-05-16T10:00:42.104149Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:109:5: note: insert 'MOZ_FALLTHROUGH;' to silence this warning
[task 2017-05-16T10:00:42.110414Z] 10:00:42     INFO -      case 1:
[task 2017-05-16T10:00:42.110470Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.110501Z] 10:00:42     INFO -      MOZ_FALLTHROUGH;
[task 2017-05-16T10:00:42.110560Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:109:5: note: insert 'break;' to avoid fall-through
[task 2017-05-16T10:00:42.110586Z] 10:00:42     INFO -      case 1:
[task 2017-05-16T10:00:42.110605Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.110626Z] 10:00:42     INFO -      break;
[task 2017-05-16T10:00:42.110689Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:115:5: error: unannotated fall-through between switch labels [-Werror,-Wimplicit-fallthrough]
[task 2017-05-16T10:00:42.110713Z] 10:00:42     INFO -      case 2:
[task 2017-05-16T10:00:42.110732Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.110784Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:115:5: note: insert 'MOZ_FALLTHROUGH;' to silence this warning
[task 2017-05-16T10:00:42.110807Z] 10:00:42     INFO -      case 2:
[task 2017-05-16T10:00:42.110826Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.110849Z] 10:00:42     INFO -      MOZ_FALLTHROUGH;
[task 2017-05-16T10:00:42.110901Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:115:5: note: insert 'break;' to avoid fall-through
[task 2017-05-16T10:00:42.110925Z] 10:00:42     INFO -      case 2:
[task 2017-05-16T10:00:42.110944Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.110965Z] 10:00:42     INFO -      break;
[task 2017-05-16T10:00:42.111028Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:131:9: error: unannotated fall-through between switch labels [-Werror,-Wimplicit-fallthrough]
[task 2017-05-16T10:00:42.111051Z] 10:00:42     INFO -          case 1:
[task 2017-05-16T10:00:42.111071Z] 10:00:42     INFO -          ^
[task 2017-05-16T10:00:42.115229Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:131:9: note: insert 'MOZ_FALLTHROUGH;' to silence this warning
[task 2017-05-16T10:00:42.115290Z] 10:00:42     INFO -          case 1:
[task 2017-05-16T10:00:42.115320Z] 10:00:42     INFO -          ^
[task 2017-05-16T10:00:42.115357Z] 10:00:42     INFO -          MOZ_FALLTHROUGH;
[task 2017-05-16T10:00:42.115416Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:131:9: note: insert 'break;' to avoid fall-through
[task 2017-05-16T10:00:42.115444Z] 10:00:42     INFO -          case 1:
[task 2017-05-16T10:00:42.115467Z] 10:00:42     INFO -          ^
[task 2017-05-16T10:00:42.115491Z] 10:00:42     INFO -          break;
[task 2017-05-16T10:00:42.115557Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:122:5: error: unannotated fall-through between switch labels [-Werror,-Wimplicit-fallthrough]
[task 2017-05-16T10:00:42.115586Z] 10:00:42     INFO -      default:
[task 2017-05-16T10:00:42.115611Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.116507Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:122:5: note: insert 'MOZ_FALLTHROUGH;' to silence this warning
[task 2017-05-16T10:00:42.116817Z] 10:00:42     INFO -      default:
[task 2017-05-16T10:00:42.117127Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.117422Z] 10:00:42     INFO -      MOZ_FALLTHROUGH;
[task 2017-05-16T10:00:42.117777Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:122:5: note: insert 'break;' to avoid fall-through
[task 2017-05-16T10:00:42.118069Z] 10:00:42     INFO -      default:
[task 2017-05-16T10:00:42.118347Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.118633Z] 10:00:42     INFO -      break;
[task 2017-05-16T10:00:42.118959Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:109:5: error: unannotated fall-through between switch labels [-Werror,-Wimplicit-fallthrough]
[task 2017-05-16T10:00:42.119253Z] 10:00:42     INFO -      case 1:
[task 2017-05-16T10:00:42.119537Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.119882Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:401:3: note: in instantiation of function template specialization 'mozilla::ipc::FuzzIntegralType<unsigned short>' requested here
[task 2017-05-16T10:00:42.120178Z] 10:00:42     INFO -    FuzzIntegralType<uint16_t>(aValue, true);
[task 2017-05-16T10:00:42.120477Z] 10:00:42     INFO -    ^
[task 2017-05-16T10:00:42.121325Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:109:5: note: insert 'MOZ_FALLTHROUGH;' to silence this warning
[task 2017-05-16T10:00:42.123459Z] 10:00:42     INFO -      case 1:
[task 2017-05-16T10:00:42.123783Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.124104Z] 10:00:42     INFO -      MOZ_FALLTHROUGH;
[task 2017-05-16T10:00:42.124459Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:109:5: note: insert 'break;' to avoid fall-through
[task 2017-05-16T10:00:42.124794Z] 10:00:42     INFO -      case 1:
[task 2017-05-16T10:00:42.125124Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.125439Z] 10:00:42     INFO -      break;
[task 2017-05-16T10:00:42.125794Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:115:5: error: unannotated fall-through between switch labels [-Werror,-Wimplicit-fallthrough]
[task 2017-05-16T10:00:42.126113Z] 10:00:42     INFO -      case 2:
[task 2017-05-16T10:00:42.126431Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.132240Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:115:5: note: insert 'MOZ_FALLTHROUGH;' to silence this warning
[task 2017-05-16T10:00:42.132569Z] 10:00:42     INFO -      case 2:
[task 2017-05-16T10:00:42.132851Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.133182Z] 10:00:42     INFO -      MOZ_FALLTHROUGH;
[task 2017-05-16T10:00:42.133487Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:115:5: note: insert 'break;' to avoid fall-through
[task 2017-05-16T10:00:42.133774Z] 10:00:42     INFO -      case 2:
[task 2017-05-16T10:00:42.134044Z] 10:00:42     INFO -      ^
[task 2017-05-16T10:00:42.134331Z] 10:00:42     INFO -      break;
[task 2017-05-16T10:00:42.134667Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:131:9: error: unannotated fall-through between switch labels [-Werror,-Wimplicit-fallthrough]
[task 2017-05-16T10:00:42.134980Z] 10:00:42     INFO -          case 1:
[task 2017-05-16T10:00:42.135306Z] 10:00:42     INFO -          ^
[task 2017-05-16T10:00:42.140286Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:131:9: note: insert 'MOZ_FALLTHROUGH;' to silence this warning
[task 2017-05-16T10:00:42.140621Z] 10:00:42     INFO -          case 1:
[task 2017-05-16T10:00:42.140933Z] 10:00:42     INFO -          ^
[task 2017-05-16T10:00:42.141314Z] 10:00:42     INFO -          MOZ_FALLTHROUGH;
[task 2017-05-16T10:00:42.141666Z] 10:00:42     INFO -  /home/worker/workspace/build/src/ipc/glue/Faulty.cpp:131:9: note: insert 'break;' to avoid fall-through
[task 2017-05-16T10:00:42.141983Z] 10:00:42     INFO -          case 1:
[task 2017-05-16T10:00:42.143769Z] 10:00:42     INFO -          ^
[task 2017-05-16T10:00:42.144072Z] 10:00:42     INFO -          break;
[task 2017-05-16T10:00:42.144378Z] 10:00:42     INFO -  fatal error: too many errors emitted, stopping now [-ferror-limit=]
[task 2017-05-16T10:00:42.152549Z] 10:00:42     INFO -  20 errors generated.
[task 2017-05-16T10:00:42.157934Z] 10:00:42     INFO -  /home/worker/workspace/build/src/config/rules.mk:1059: recipe for target 'Faulty.o' failed
[task 2017-05-16T10:00:42.158005Z] 10:00:42     INFO -  gmake[5]: *** [Faulty.o] Error 1
[task 2017-05-16T10:00:42.158060Z] 10:00:42     INFO -  gmake[5]: Leaving directory '/home/worker/workspace/build/src/obj-firefox/ipc/glue'
[task 2017-05-16T10:00:42.158117Z] 10:00:42     INFO -  /home/worker/workspace/build/src/config/recurse.mk:73: recipe for target 'ipc/glue/target' failed
[task 2017-05-16T10:00:42.158154Z] 10:00:42     INFO -  gmake[4]: *** [ipc/glue/target] Error 2
[task 2017-05-16T10:00:42.158190Z] 10:00:42     INFO -  gmake[4]: *** Waiting for unfinished jobs....
[task 2017-05-16T10:00:42.166361Z] 10:00:42     INFO -  /home/worker/workspace/build/src/clang/bin/clang++ -B /home/worker/workspace/build/src/gcc/bin -std=gnu++11 -o UnifiedProtocols5.o -c -I/home/worker/workspace/build/src/obj-firefox/
Just keep in mind, those were warnings. The build aborted cause of errors, which occurred already before: "fatal error: too many errors emitted". 
Though if you fix those warnings, that would be of great help!
Comment hidden (mozreview-request)
Comment on attachment 8868906 [details]
Bug 1365321 - remove warnings from faulty

Oh, I see those are indeed errors now.

Thanks rforbes!
Attachment #8868906 - Flags: feedback+
(Assignee)

Updated

a year ago
Attachment #8868906 - Flags: review?(wmccloskey)
Comment on attachment 8868906 [details]
Bug 1365321 - remove warnings from faulty

https://reviewboard.mozilla.org/r/140536/#review144732
Attachment #8868906 - Flags: review?(wmccloskey) → review+

Comment 5

a year ago
hg error in cmd: hg push -r tip ssh://hg.mozilla.org/integration/autoland: pushing to ssh://hg.mozilla.org/integration/autoland
searching for changes
remote: adding changesets
remote: adding manifests
remote: adding file changes
remote: added 1 changesets with 1 changes to 1 files
remote: 
remote: 
remote: ************************** ERROR ****************************
remote: Rev fd3f0ca8b8e8 needs "Bug N" or "No bug" in the commit message.
remote: Raymond Forbes <rforbes@mozilla.com>
remote: 1365321 - Remove warnings from faulty r=billm
remote: 
remote: MozReview-Commit-ID: HpK0oLEaQbR
remote: *************************************************************
remote: 
remote: 
remote: transaction abort!
remote: rollback completed
remote: pretxnchangegroup.c_commitmessage hook failed
abort: push failed on remote

Comment 6

a year ago
hg error in cmd: hg push -r tip ssh://hg.mozilla.org/integration/autoland: pushing to ssh://hg.mozilla.org/integration/autoland
searching for changes
remote: adding changesets
remote: adding manifests
remote: adding file changes
remote: added 1 changesets with 1 changes to 1 files
remote: 
remote: 
remote: ************************** ERROR ****************************
remote: Rev fd3f0ca8b8e8 needs "Bug N" or "No bug" in the commit message.
remote: Raymond Forbes <rforbes@mozilla.com>
remote: 1365321 - Remove warnings from faulty r=billm
remote: 
remote: MozReview-Commit-ID: HpK0oLEaQbR
remote: *************************************************************
remote: 
remote: 
remote: transaction abort!
remote: rollback completed
remote: pretxnchangegroup.c_commitmessage hook failed
abort: push failed on remote
Comment hidden (mozreview-request)

Comment 8

a year ago
Pushed by choller@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/adb2e187d9ac
remove warnings from faulty r=billm

Comment 9

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/adb2e187d9ac
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
Assignee: nobody → rforbes
You need to log in before you can comment on or make changes to this bug.