Issue #2343 - replace MOZ_FALLTHROUGH with [[fallthrough]]
Basically a S&R. Removed the macro and adjusts IDL codegen accordingly.
This commit is contained in:
138 files changed
+299
-343
No files matched your search
@@ -135,7 +135,7 @@ TimingParams::ParseEasing(const nsAString& aEasing,
|
||||
NS_STYLE_TRANSITION_TIMING_FUNCTION_LINEAR) {
|
||||
return Nothing();
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eCSSUnit_Cubic_Bezier:
|
||||
case eCSSUnit_Steps: {
|
||||
nsTimingFunction timingFunction;
|
||||
|
||||
@@ -3199,7 +3199,7 @@ Element::GetEventTargetParentForLinks(EventChainPreVisitor& aVisitor)
|
||||
// Set the status bar similarly for mouseover and focus
|
||||
case eMouseOver:
|
||||
aVisitor.mEventStatus = nsEventStatus_eConsumeNoDefault;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eFocus: {
|
||||
InternalFocusEvent* focusEvent = aVisitor.mEvent->AsFocusEvent();
|
||||
if (!focusEvent || !focusEvent->mIsRefocus) {
|
||||
@@ -3214,7 +3214,7 @@ Element::GetEventTargetParentForLinks(EventChainPreVisitor& aVisitor)
|
||||
}
|
||||
case eMouseOut:
|
||||
aVisitor.mEventStatus = nsEventStatus_eConsumeNoDefault;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eBlur:
|
||||
rv = LeaveLink(aVisitor.mPresContext);
|
||||
if (NS_SUCCEEDED(rv)) {
|
||||
|
||||
@@ -5787,7 +5787,7 @@ nsIDocument::ImportNode(nsINode& aNode, bool aDeep, ErrorResult& rv) const
|
||||
if (ShadowRoot::FromNode(imported)) {
|
||||
break;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
}
|
||||
case nsIDOMNode::ATTRIBUTE_NODE:
|
||||
case nsIDOMNode::ELEMENT_NODE:
|
||||
@@ -6893,7 +6893,7 @@ nsIDocument::AdoptNode(nsINode& aAdoptedNode, ErrorResult& rv)
|
||||
rv.Throw(NS_ERROR_DOM_HIERARCHY_REQUEST_ERR);
|
||||
return nullptr;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
}
|
||||
case nsIDOMNode::ELEMENT_NODE:
|
||||
case nsIDOMNode::PROCESSING_INSTRUCTION_NODE:
|
||||
@@ -7184,7 +7184,7 @@ nsDocument::GetViewportInfo(const ScreenIntSize& aDisplaySize)
|
||||
mValidMaxScale = !maxScaleStr.IsEmpty() && NS_SUCCEEDED(scaleMaxErrorCode);
|
||||
|
||||
mViewportType = Specified;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
}
|
||||
case Specified:
|
||||
default:
|
||||
|
||||
@@ -1442,7 +1442,7 @@ nsXMLContentSerializer::AppendFormatedWrapped_WhitespaceSequence(
|
||||
case ' ':
|
||||
case '\t':
|
||||
sawBlankOrTab = true;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case '\n':
|
||||
++aPos;
|
||||
// do not increase mColPos,
|
||||
|
||||
@@ -7918,7 +7918,7 @@ class CGCase(CGList):
|
||||
self.append(CGGeneric("case " + expression + ": {\n"))
|
||||
bodyList = CGList([body])
|
||||
if fallThrough:
|
||||
bodyList.append(CGGeneric("MOZ_FALLTHROUGH;\n"))
|
||||
bodyList.append(CGGeneric("[[fallthrough]];\n"))
|
||||
else:
|
||||
bodyList.append(CGGeneric("break;\n"))
|
||||
self.append(CGIndenter(bodyList))
|
||||
|
||||
@@ -121,7 +121,7 @@ WebGL2Context::GetParameter(JSContext* cx, GLenum pname, ErrorResult& rv)
|
||||
return JS::NumberValue(UINT32_MAX);
|
||||
|
||||
/*** fall through to fGetInteger64v ***/
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
case LOCAL_GL_MAX_COMBINED_FRAGMENT_UNIFORM_COMPONENTS:
|
||||
case LOCAL_GL_MAX_COMBINED_VERTEX_UNIFORM_COMPONENTS:
|
||||
|
||||
@@ -792,7 +792,7 @@ WebGLContext::GetRenderbufferParameter(GLenum target, GLenum pname)
|
||||
case LOCAL_GL_RENDERBUFFER_SAMPLES:
|
||||
if (!IsWebGL2())
|
||||
break;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
case LOCAL_GL_RENDERBUFFER_WIDTH:
|
||||
case LOCAL_GL_RENDERBUFFER_HEIGHT:
|
||||
|
||||
@@ -178,7 +178,7 @@ WebGLQuery::GetQueryParameter(GLenum pname, JS::MutableHandleValue retval) const
|
||||
gl->fGetQueryObjectui64v(mGLName, pname, &val);
|
||||
break;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
default:
|
||||
gl->fGetQueryObjectuiv(mGLName, LOCAL_GL_QUERY_RESULT, (GLuint*)&val);
|
||||
|
||||
@@ -654,7 +654,7 @@ EventStateManager::PreHandleEvent(nsPresContext* aPresContext,
|
||||
StopTrackingDragGesture();
|
||||
sNormalLMouseEventInProcess = false;
|
||||
// then fall through...
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case WidgetMouseEvent::eRightButton:
|
||||
case WidgetMouseEvent::eMiddleButton:
|
||||
SetClickCount(mouseEvent, aStatus);
|
||||
@@ -708,7 +708,7 @@ EventStateManager::PreHandleEvent(nsPresContext* aPresContext,
|
||||
aEvent->mMessage = eVoidEvent;
|
||||
break;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eMouseMove:
|
||||
case ePointerDown:
|
||||
case ePointerMove: {
|
||||
@@ -774,7 +774,7 @@ EventStateManager::PreHandleEvent(nsPresContext* aPresContext,
|
||||
}
|
||||
}
|
||||
// then fall through...
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eBeforeKeyDown:
|
||||
case eKeyDown:
|
||||
case eAfterKeyDown:
|
||||
@@ -3153,7 +3153,7 @@ EventStateManager::PostHandleEvent(nsPresContext* aPresContext,
|
||||
}
|
||||
// After firing the pointercancel event, a user agent must also fire a
|
||||
// pointerout event followed by a pointerleave event.
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
}
|
||||
case ePointerUp: {
|
||||
WidgetPointerEvent* pointerEvent = aEvent->AsPointerEvent();
|
||||
@@ -4272,7 +4272,7 @@ EventStateManager::GenerateMouseEnterExit(WidgetMouseEvent* aMouseEvent)
|
||||
// Update the last known mRefPoint with the current mRefPoint.
|
||||
sLastRefPoint = aMouseEvent->mRefPoint;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case ePointerMove:
|
||||
case ePointerDown:
|
||||
case ePointerGotCapture:
|
||||
|
||||
@@ -4545,7 +4545,7 @@ HTMLInputElement::PostHandleEvent(EventChainPostVisitor& aVisitor)
|
||||
break; // If we are submitting, do not send click event
|
||||
}
|
||||
// else fall through and treat Space like click...
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
}
|
||||
case NS_FORM_INPUT_BUTTON:
|
||||
case NS_FORM_INPUT_RESET:
|
||||
@@ -4567,7 +4567,7 @@ HTMLInputElement::PostHandleEvent(EventChainPostVisitor& aVisitor)
|
||||
case NS_VK_UP:
|
||||
case NS_VK_LEFT:
|
||||
isMovingBack = true;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case NS_VK_DOWN:
|
||||
case NS_VK_RIGHT:
|
||||
// Arrow key pressed, focus+select prev/next radio button
|
||||
|
||||
@@ -144,9 +144,9 @@ AudioConfig::SampleSize(AudioConfig::SampleFormat aFormat)
|
||||
switch (aFormat) {
|
||||
case FORMAT_U8: return 1;
|
||||
case FORMAT_S16: return 2;
|
||||
case FORMAT_S24: MOZ_FALLTHROUGH;
|
||||
case FORMAT_S24LSB: MOZ_FALLTHROUGH;
|
||||
case FORMAT_S32: MOZ_FALLTHROUGH;
|
||||
case FORMAT_S24: [[fallthrough]];
|
||||
case FORMAT_S24LSB: [[fallthrough]];
|
||||
case FORMAT_S32: [[fallthrough]];
|
||||
case FORMAT_FLT: return 4;
|
||||
case FORMAT_NONE:
|
||||
default: return 0;
|
||||
@@ -159,11 +159,11 @@ AudioConfig::FormatToBits(AudioConfig::SampleFormat aFormat)
|
||||
switch (aFormat) {
|
||||
case FORMAT_U8: return 8;
|
||||
case FORMAT_S16: return 16;
|
||||
case FORMAT_S24LSB: MOZ_FALLTHROUGH;
|
||||
case FORMAT_S24LSB: [[fallthrough]];
|
||||
case FORMAT_S24: return 24;
|
||||
case FORMAT_S32: MOZ_FALLTHROUGH;
|
||||
case FORMAT_S32: [[fallthrough]];
|
||||
case FORMAT_FLT: return 32;
|
||||
case FORMAT_NONE: MOZ_FALLTHROUGH;
|
||||
case FORMAT_NONE: [[fallthrough]];
|
||||
default: return 0;
|
||||
}
|
||||
}
|
||||
|
||||
@@ -2053,7 +2053,7 @@ MediaManager::GetUserMedia(nsPIDOMWindowInner* aWindow,
|
||||
nsPIDOMWindowOuter* outer = aWindow->GetOuterWindow();
|
||||
vc.mBrowserWindow.Construct(outer->WindowID());
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case MediaSourceEnum::Screen:
|
||||
case MediaSourceEnum::Application:
|
||||
case MediaSourceEnum::Window:
|
||||
|
||||
@@ -263,7 +263,7 @@ MP4TrackDemuxer::GetNextSample()
|
||||
mp4_demuxer::H264::FrameType type =
|
||||
mp4_demuxer::H264::GetFrameType(sample);
|
||||
switch (type) {
|
||||
case mp4_demuxer::H264::FrameType::I_FRAME: MOZ_FALLTHROUGH;
|
||||
case mp4_demuxer::H264::FrameType::I_FRAME: [[fallthrough]];
|
||||
case mp4_demuxer::H264::FrameType::OTHER:
|
||||
{
|
||||
bool keyframe = type == mp4_demuxer::H264::FrameType::I_FRAME;
|
||||
|
||||
@@ -141,7 +141,7 @@ bool WebMBufferedParser::Append(const unsigned char* aBuffer, uint32_t aLength,
|
||||
case EBML_ID:
|
||||
mLastInitStartOffset = mCurrentOffset + (p - aBuffer) -
|
||||
(mElement.mID.mLength + mElement.mSize.mLength);
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
default:
|
||||
mSkipBytes = mElement.mSize.mValue;
|
||||
mState = SKIP_DATA;
|
||||
|
||||
@@ -2022,7 +2022,7 @@ _getvalue(NPP npp, NPNVariable variable, void *result)
|
||||
// old XPCOM objects, no longer supported, but null out the out
|
||||
// param to avoid crashing plugins that still try to use this.
|
||||
*(nsISupports**)result = nullptr;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
default:
|
||||
NPN_PLUGIN_LOG(PLUGIN_LOG_NORMAL, ("NPN_getvalue unhandled get value: %d\n", variable));
|
||||
|
||||
@@ -561,7 +561,7 @@ PluginInstanceChild::NPN_GetValue(NPNVariable aVar,
|
||||
case NPNVSupportsXEmbedBool:
|
||||
case NPNVSupportsWindowless:
|
||||
NS_NOTREACHED("NPNVariable should be handled in PluginModuleChild.");
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
#endif
|
||||
|
||||
default:
|
||||
|
||||
@@ -186,7 +186,7 @@ ParseClockValue(RangedPtr<const char16_t>& aIter,
|
||||
!ParseColon(iter, aEnd)) {
|
||||
return false;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case PARTIAL_CLOCK_VALUE:
|
||||
if (!ParseSecondsOrMinutes(iter, aEnd, minutes) ||
|
||||
!ParseColon(iter, aEnd) ||
|
||||
|
||||
@@ -284,7 +284,7 @@ DOMStorageDBThread::InsertDBOp(DOMStorageDBThread::DBOperation* aOperation)
|
||||
aOperation->Finalize(NS_OK);
|
||||
return NS_OK;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
case DBOperation::opGetUsage:
|
||||
if (aOperation->Type() == DBOperation::opPreloadUrgent) {
|
||||
|
||||
@@ -385,7 +385,7 @@ nsresult Update(mozIStorageConnection *aWorkerConnection)
|
||||
rv = aWorkerConnection->SetSchemaVersion(1);
|
||||
NS_ENSURE_SUCCESS(rv, rv);
|
||||
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
}
|
||||
case CURRENT_SCHEMA_VERSION:
|
||||
// Ensure the tables and indexes are up. This is mostly a no-op
|
||||
|
||||
@@ -351,7 +351,7 @@ ConvertPathSegmentData(SVGPathDataAndInfo::const_iterator& aStart,
|
||||
aResult[5] = aStart[5];
|
||||
aResult[6] = aStart[6];
|
||||
AdjustSegmentForRelativeness(adjustmentType, aResult + 5, aState);
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case PATHSEG_CURVETO_QUADRATIC_ABS:
|
||||
case PATHSEG_CURVETO_QUADRATIC_REL:
|
||||
case PATHSEG_CURVETO_CUBIC_SMOOTH_ABS:
|
||||
@@ -359,7 +359,7 @@ ConvertPathSegmentData(SVGPathDataAndInfo::const_iterator& aStart,
|
||||
aResult[3] = aStart[3];
|
||||
aResult[4] = aStart[4];
|
||||
AdjustSegmentForRelativeness(adjustmentType, aResult + 3, aState);
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case PATHSEG_MOVETO_ABS:
|
||||
case PATHSEG_MOVETO_REL:
|
||||
case PATHSEG_LINETO_ABS:
|
||||
|
||||
@@ -142,7 +142,7 @@ SVGTransformListParser::ParseTranslate()
|
||||
switch (count) {
|
||||
case 1:
|
||||
t[1] = 0.f;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case 2:
|
||||
{
|
||||
nsSVGTransform* transform = mTransforms.AppendElement(fallible);
|
||||
@@ -170,7 +170,7 @@ SVGTransformListParser::ParseScale()
|
||||
switch (count) {
|
||||
case 1:
|
||||
s[1] = s[0];
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case 2:
|
||||
{
|
||||
nsSVGTransform* transform = mTransforms.AppendElement(fallible);
|
||||
@@ -199,7 +199,7 @@ SVGTransformListParser::ParseRotate()
|
||||
switch (count) {
|
||||
case 1:
|
||||
r[1] = r[2] = 0.f;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case 3:
|
||||
{
|
||||
nsSVGTransform* transform = mTransforms.AppendElement(fallible);
|
||||
|
||||
@@ -255,7 +255,7 @@ protected:
|
||||
if (!IsNaN(gpsData.fix.altitude)) {
|
||||
alt = gpsData.fix.altitude;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case MODE_2D:
|
||||
if (!IsNaN(gpsData.fix.latitude)) {
|
||||
lat = gpsData.fix.latitude;
|
||||
@@ -308,13 +308,13 @@ protected:
|
||||
{
|
||||
switch (aErrno) {
|
||||
case EACCES:
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case EPERM:
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case EROFS:
|
||||
return nsIDOMGeoPositionError::PERMISSION_DENIED;
|
||||
case ETIME:
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case ETIMEDOUT:
|
||||
return nsIDOMGeoPositionError::TIMEOUT;
|
||||
default:
|
||||
|
||||
@@ -42,7 +42,7 @@ LocationStep::evaluate(txIEvalContext* aContext, txAExprResult** aResult)
|
||||
if (!walker.moveToParent()) {
|
||||
break;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
}
|
||||
case ANCESTOR_OR_SELF_AXIS:
|
||||
{
|
||||
@@ -74,7 +74,7 @@ LocationStep::evaluate(txIEvalContext* aContext, txAExprResult** aResult)
|
||||
if (mNodeTest->matches(walker.getCurrentPosition(), aContext)) {
|
||||
nodes->append(walker.getCurrentPosition());
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
}
|
||||
case DESCENDANT_AXIS:
|
||||
{
|
||||
|
||||
@@ -81,7 +81,7 @@ void txOutputFormat::setFromDefaults()
|
||||
case eMethodNotSet:
|
||||
{
|
||||
mMethod = eXMLOutput;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
}
|
||||
case eXMLOutput:
|
||||
{
|
||||
|
||||
@@ -190,7 +190,7 @@ nsCookiePermission::CanSetCookie(nsIURI *aURI,
|
||||
switch (perm) {
|
||||
case nsICookiePermission::ACCESS_SESSION:
|
||||
*aIsSession = true;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
case nsICookiePermission::ACCESS_ALLOW:
|
||||
*aResult = true;
|
||||
|
||||
@@ -935,7 +935,7 @@ nsPermissionManager::InitDB(bool aRemoveFile)
|
||||
}
|
||||
|
||||
// fall through to the next upgrade
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
// TODO: we want to make default version as version 2 in order to fix bug 784875.
|
||||
case 0:
|
||||
@@ -955,7 +955,7 @@ nsPermissionManager::InitDB(bool aRemoveFile)
|
||||
}
|
||||
|
||||
// fall through to the next upgrade
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
// Version 3->4 is the creation of the modificationTime field.
|
||||
case 3:
|
||||
@@ -973,7 +973,7 @@ nsPermissionManager::InitDB(bool aRemoveFile)
|
||||
}
|
||||
|
||||
// fall through to the next upgrade
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
// In version 5, host appId, and isInBrowserElement were merged into a
|
||||
// single origin entry
|
||||
@@ -1059,7 +1059,7 @@ nsPermissionManager::InitDB(bool aRemoveFile)
|
||||
}
|
||||
|
||||
// fall through to the next upgrade
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
// At this point, the version 5 table has been migrated to a version 6 table
|
||||
// We are guaranteed to have at least one of moz_hosts and moz_perms. If
|
||||
@@ -1216,7 +1216,7 @@ nsPermissionManager::InitDB(bool aRemoveFile)
|
||||
}
|
||||
|
||||
// fall through to the next upgrade
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
// The version 7-8 migration is the re-migration of localhost and ip-address
|
||||
// entries due to errors in the previous version 7 migration which caused
|
||||
@@ -1318,7 +1318,7 @@ nsPermissionManager::InitDB(bool aRemoveFile)
|
||||
}
|
||||
|
||||
// fall through to the next upgrade
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
// The version 8-9 migration removes the unnecessary backup moz-hosts database contents.
|
||||
// as the data no longer needs to be migrated
|
||||
@@ -1346,7 +1346,7 @@ nsPermissionManager::InitDB(bool aRemoveFile)
|
||||
}
|
||||
|
||||
// fall through to the next upgrade
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
// current version.
|
||||
case HOSTS_SCHEMA_VERSION:
|
||||
|
||||
@@ -1089,7 +1089,7 @@ nsEventStatus AsyncPanZoomController::OnTouchStart(const MultiTouchInput& aEvent
|
||||
case PAN_MOMENTUM:
|
||||
MOZ_ASSERT(GetCurrentTouchBlock());
|
||||
GetCurrentTouchBlock()->GetOverscrollHandoffChain()->CancelAnimations(ExcludeOverscroll);
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case NOTHING: {
|
||||
mX.StartTouch(point.x, aEvent.mTime);
|
||||
mY.StartTouch(point.y, aEvent.mTime);
|
||||
@@ -1196,7 +1196,7 @@ nsEventStatus AsyncPanZoomController::OnTouchEnd(const MultiTouchInput& aEvent)
|
||||
case FLING:
|
||||
// Should never happen.
|
||||
NS_WARNING("Received impossible touch end in OnTouchEnd.");
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case ANIMATING_ZOOM:
|
||||
case SMOOTH_SCROLL:
|
||||
case NOTHING:
|
||||
|
||||
@@ -438,7 +438,7 @@ void GestureEventListener::HandleInputTimeoutLongTap()
|
||||
// just in case MAX_TAP_TIME > ContextMenuDelay cancel MAX_TAP timer
|
||||
// and fall through
|
||||
CancelMaxTapTimeoutTask();
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case GESTURE_FIRST_SINGLE_TOUCH_MAX_TAP_DOWN: {
|
||||
SetState(GESTURE_LONG_TOUCH_DOWN);
|
||||
mAsyncPanZoomController->HandleGestureEvent(
|
||||
|
||||
@@ -347,10 +347,10 @@ APZEventState::ProcessTouchEvent(const WidgetTouchEvent& aEvent,
|
||||
mTouchEndCancelled = true;
|
||||
mEndTouchIsClick = false;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eTouchCancel:
|
||||
mActiveElementManager->HandleTouchEndEvent(mEndTouchIsClick);
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eTouchMove: {
|
||||
if (mPendingTouchPreventedResponse) {
|
||||
MOZ_ASSERT(aGuid == mPendingTouchPreventedGuid);
|
||||
|
||||
@@ -625,7 +625,7 @@ PrepareFontOptions(FcPattern* aPattern,
|
||||
// subpixel_order won't be used by the font as we won't use
|
||||
// CAIRO_ANTIALIAS_SUBPIXEL, but don't leave it at default for
|
||||
// caching reasons described above. Fall through:
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case FC_RGBA_RGB:
|
||||
subpixel_order = CAIRO_SUBPIXEL_ORDER_RGB;
|
||||
break;
|
||||
|
||||
@@ -479,7 +479,7 @@ gfxFontShaper::MergeFontFeatures(
|
||||
case NS_FONT_VARIANT_CAPS_ALLSMALL:
|
||||
mergedFeatures.Put(HB_TAG('c','2','s','c'), 1);
|
||||
// fall through to the small-caps case
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
case NS_FONT_VARIANT_CAPS_SMALLCAPS:
|
||||
mergedFeatures.Put(HB_TAG('s','m','c','p'), 1);
|
||||
@@ -489,7 +489,7 @@ gfxFontShaper::MergeFontFeatures(
|
||||
mergedFeatures.Put(aAddSmallCaps ? HB_TAG('c','2','s','c') :
|
||||
HB_TAG('c','2','p','c'), 1);
|
||||
// fall through to the petite-caps case
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
case NS_FONT_VARIANT_CAPS_PETITECAPS:
|
||||
mergedFeatures.Put(aAddSmallCaps ? HB_TAG('s','m','c','p') :
|
||||
@@ -615,7 +615,7 @@ gfxFontShaper::GetRoundOffsetsToPixels(DrawTarget* aDrawTarget,
|
||||
DWRITE_MEASURING_MODE_NATURAL) {
|
||||
return;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
#endif
|
||||
case CAIRO_FONT_TYPE_QUARTZ:
|
||||
// Quartz surfaces implement show_glyphs for Quartz fonts
|
||||
@@ -3119,7 +3119,7 @@ gfxFont::InitFakeSmallCapsRun(DrawTarget *aDrawTarget,
|
||||
case kUppercaseReduce:
|
||||
// use reduced-size font, then fall through to uppercase the text
|
||||
f = smallCapsFont;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
case kUppercase:
|
||||
// apply uppercase transform to the string
|
||||
|
||||
@@ -1758,7 +1758,7 @@ gfxFontGroup::FamilyFace::CheckState(bool& aSkipDrawing)
|
||||
case gfxUserFontEntry::STATUS_FAILED:
|
||||
SetInvalid();
|
||||
// fall-thru to the default case
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
default:
|
||||
SetLoading(false);
|
||||
}
|
||||
|
||||
@@ -412,7 +412,7 @@ nsJPEGDecoder::ReadJPEGData(const char* aData, size_t aLength)
|
||||
mInfo.output_width, mInfo.output_height));
|
||||
|
||||
mState = JPEG_START_DECOMPRESS;
|
||||
MOZ_FALLTHROUGH; // to start decompressing.
|
||||
[[fallthrough]]; // to start decompressing.
|
||||
}
|
||||
|
||||
case JPEG_START_DECOMPRESS: {
|
||||
@@ -439,7 +439,7 @@ nsJPEGDecoder::ReadJPEGData(const char* aData, size_t aLength)
|
||||
// If this is a progressive JPEG ...
|
||||
mState = mInfo.buffered_image ?
|
||||
JPEG_DECOMPRESS_PROGRESSIVE : JPEG_DECOMPRESS_SEQUENTIAL;
|
||||
MOZ_FALLTHROUGH; // to decompress sequential JPEG.
|
||||
[[fallthrough]]; // to decompress sequential JPEG.
|
||||
}
|
||||
|
||||
case JPEG_DECOMPRESS_SEQUENTIAL: {
|
||||
@@ -461,7 +461,7 @@ nsJPEGDecoder::ReadJPEGData(const char* aData, size_t aLength)
|
||||
"We didn't process all of the data!");
|
||||
mState = JPEG_DONE;
|
||||
}
|
||||
MOZ_FALLTHROUGH; // to decompress progressive JPEG.
|
||||
[[fallthrough]]; // to decompress progressive JPEG.
|
||||
}
|
||||
|
||||
case JPEG_DECOMPRESS_PROGRESSIVE: {
|
||||
@@ -534,7 +534,7 @@ nsJPEGDecoder::ReadJPEGData(const char* aData, size_t aLength)
|
||||
|
||||
mState = JPEG_DONE;
|
||||
}
|
||||
MOZ_FALLTHROUGH; // to finish decompressing.
|
||||
[[fallthrough]]; // to finish decompressing.
|
||||
}
|
||||
|
||||
case JPEG_DONE: {
|
||||
|
||||
@@ -336,7 +336,7 @@ NS_IMETHODIMP nsEUCJPToUnicodeV2::Convert(
|
||||
break;
|
||||
}
|
||||
// else fall through to error handler
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
}
|
||||
case 5: // two bytes undefined
|
||||
{
|
||||
|
||||
@@ -2537,7 +2537,7 @@ ASTSerializer::statement(ParseNode* pn, MutableHandleValue dst)
|
||||
pn = pn->as<LexicalScopeNode>().scopeBody();
|
||||
if (!pn->isKind(PNK_STATEMENTLIST))
|
||||
return statement(pn, dst);
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
case PNK_STATEMENTLIST:
|
||||
return blockStatement(&pn->as<ListNode>(), dst);
|
||||
|
||||
@@ -4519,7 +4519,7 @@ CType::Finalize(JSFreeOp* fop, JSObject* obj)
|
||||
}
|
||||
}
|
||||
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
case TYPE_array: {
|
||||
// Free the ffi_type info.
|
||||
|
||||
@@ -1271,7 +1271,7 @@ BytecodeEmitter::checkSideEffects(ParseNode* pn, bool* answer)
|
||||
// Any subexpression of a comma expression could be effectful.
|
||||
case PNK_COMMA:
|
||||
MOZ_ASSERT(!pn->as<ListNode>().empty());
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
// Subcomponents of a literal may be effectful.
|
||||
case PNK_ARRAY:
|
||||
case PNK_OBJECT:
|
||||
|
||||
@@ -174,7 +174,7 @@ class NameResolver
|
||||
* flagged as a contributor.
|
||||
*/
|
||||
pos--;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
default:
|
||||
/* Save any other nodes we encounter on the way up. */
|
||||
|
||||
@@ -1544,12 +1544,12 @@ Parser<ParseHandler>::noteDeclaredName(HandlePropertyName name, DeclarationKind
|
||||
return false;
|
||||
}
|
||||
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
case DeclarationKind::Import:
|
||||
// Module code is always strict, so 'let' is always a keyword and never a name.
|
||||
MOZ_ASSERT(name != context->names().let);
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
case DeclarationKind::SimpleCatchParameter:
|
||||
case DeclarationKind::CatchParameter: {
|
||||
@@ -6921,7 +6921,7 @@ Parser<ParseHandler>::yieldExpression(InHandling inHandling)
|
||||
case TOK_MUL:
|
||||
kind = PNK_YIELD_STAR;
|
||||
tokenStream.consumeKnownToken(TOK_MUL, TokenStream::Operand);
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
default:
|
||||
exprNode = assignExpr(inHandling, YieldIsKeyword, TripledotProhibited);
|
||||
if (!exprNode)
|
||||
@@ -6972,7 +6972,7 @@ Parser<ParseHandler>::yieldExpression(InHandling inHandling)
|
||||
|
||||
pc->functionBox()->setGeneratorKind(LegacyGenerator);
|
||||
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
case LegacyGenerator:
|
||||
{
|
||||
@@ -9618,7 +9618,7 @@ Parser<ParseHandler>::unaryExpr(YieldHandling yieldHandling, TripledotHandling t
|
||||
}
|
||||
}
|
||||
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
default: {
|
||||
Node expr = optionalExpr(yieldHandling, tripledotHandling, tt,
|
||||
|
||||
@@ -3348,7 +3348,7 @@ EmitAtomLetter(RegExpCompiler* compiler,
|
||||
}
|
||||
case 4:
|
||||
macro_assembler->CheckCharacter(chars[3], &ok);
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case 3:
|
||||
macro_assembler->CheckCharacter(chars[0], &ok);
|
||||
macro_assembler->CheckCharacter(chars[1], &ok);
|
||||
|
||||
@@ -987,7 +987,7 @@ RegExpParser<CharT>::ParseClassEscape(char16_t* char_class, widechar *value,
|
||||
}
|
||||
return true;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
default:
|
||||
if (!ParseClassCharacterEscape(value))
|
||||
return false;
|
||||
@@ -1981,7 +1981,7 @@ RegExpParser<CharT>::ParseDisjunction()
|
||||
Advance(2);
|
||||
break;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
}
|
||||
case '0': {
|
||||
if (unicode_) {
|
||||
@@ -2094,7 +2094,7 @@ RegExpParser<CharT>::ParseDisjunction()
|
||||
int dummy;
|
||||
if (ParseIntervalQuantifier(&dummy, &dummy))
|
||||
return ReportError(JSMSG_NOTHING_TO_REPEAT);
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
}
|
||||
default:
|
||||
if (unicode_) {
|
||||
|
||||
@@ -2533,7 +2533,7 @@ BacktrackingAllocator::computeSpillWeight(LiveBundle* bundle)
|
||||
|
||||
case LUse::FIXED:
|
||||
fixed = true;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case LUse::REGISTER:
|
||||
usesTotal += 2000;
|
||||
break;
|
||||
|
||||
@@ -1975,7 +1975,7 @@ jit::FinishBailoutToBaseline(BaselineBailoutInfo* bailoutInfo)
|
||||
// Invalid assumption based on baseline code.
|
||||
case Bailout_OverflowInvalidate:
|
||||
outerScript->setHadOverflowBailout();
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case Bailout_NonStringInputInvalidate:
|
||||
case Bailout_DoubleOutput:
|
||||
case Bailout_ObjectIdentityOrTypeGuard:
|
||||
|
||||
@@ -929,7 +929,7 @@ RemoveExistingGetElemNativeStubs(JSContext* cx, ICGetElem_Fallback* stub, Handle
|
||||
case ICStub::GetElem_NativeSlotSymbol:
|
||||
if (indirect)
|
||||
continue;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case ICStub::GetElem_NativePrototypeSlotName:
|
||||
case ICStub::GetElem_NativePrototypeSlotSymbol:
|
||||
case ICStub::GetElem_NativePrototypeCallNativeName:
|
||||
|
||||
@@ -1596,7 +1596,7 @@ IonBuilder::traverseBytecode()
|
||||
MOZ_ASSERT(i == 0);
|
||||
if (current->peek(-1) == popped[0])
|
||||
break;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
default:
|
||||
MOZ_ASSERT(popped[i]->isImplicitlyUsed() ||
|
||||
@@ -2032,7 +2032,7 @@ IonBuilder::inspectOpcode(JSOp op)
|
||||
}
|
||||
}
|
||||
// Fall through to JSOP_BINDNAME
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case JSOP_BINDNAME:
|
||||
return jsop_bindname(info().getName(pc));
|
||||
|
||||
|
||||
@@ -2022,7 +2022,7 @@ LIRGenerator::visitToDouble(MToDouble* convert)
|
||||
|
||||
case MIRType::Boolean:
|
||||
MOZ_ASSERT(conversion != MToFPInstruction::NumbersOnly);
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
case MIRType::Int32:
|
||||
{
|
||||
@@ -2077,7 +2077,7 @@ LIRGenerator::visitToFloat32(MToFloat32* convert)
|
||||
|
||||
case MIRType::Boolean:
|
||||
MOZ_ASSERT(conversion != MToFPInstruction::NumbersOnly);
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
case MIRType::Int32:
|
||||
{
|
||||
|
||||
@@ -2432,7 +2432,7 @@ IonBuilder::inlineIsTypedArrayHelper(CallInfo& callInfo, WrappingBehavior wrappi
|
||||
}
|
||||
}
|
||||
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
case TemporaryTypeSet::ForAllResult::EMPTY:
|
||||
result = false;
|
||||
|
||||
+3
-3
@@ -2409,7 +2409,7 @@ CanProduceNegativeZero(MDefinition* def)
|
||||
case MDefinition::Op_Constant:
|
||||
if (def->type() == MIRType::Double && def->toConstant()->toDouble() == -0.0)
|
||||
return true;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case MDefinition::Op_BitAnd:
|
||||
case MDefinition::Op_BitOr:
|
||||
case MDefinition::Op_BitXor:
|
||||
@@ -2491,7 +2491,7 @@ NeedNegativeZeroCheck(MDefinition* def)
|
||||
if (rhs->id() < lhs->id() && CanProduceNegativeZero(lhs))
|
||||
return true;
|
||||
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
}
|
||||
case MDefinition::Op_StoreElement:
|
||||
case MDefinition::Op_StoreElementHole:
|
||||
@@ -3379,7 +3379,7 @@ MTypeOf::foldsTo(TempAllocator& alloc)
|
||||
type = JSTYPE_OBJECT;
|
||||
break;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
default:
|
||||
return this;
|
||||
}
|
||||
|
||||
@@ -405,7 +405,7 @@ MacroAssembler::loadUnboxedProperty(T address, JSValueType type, TypedOrValueReg
|
||||
convertInt32ToDouble(address, output.typedReg().fpu());
|
||||
break;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
}
|
||||
|
||||
case JSVAL_TYPE_BOOLEAN:
|
||||
|
||||
@@ -276,7 +276,7 @@ RangeAnalysis::addBetaNodes()
|
||||
if (!compare->isNumericComparison())
|
||||
continue;
|
||||
// Otherwise fall through to handle JSOP_STRICTEQ the same as JSOP_EQ.
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case JSOP_EQ:
|
||||
comp.setDouble(bound, bound);
|
||||
break;
|
||||
@@ -285,7 +285,7 @@ RangeAnalysis::addBetaNodes()
|
||||
if (!compare->isNumericComparison())
|
||||
continue;
|
||||
// Otherwise fall through to handle JSOP_STRICTNE the same as JSOP_NE.
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case JSOP_NE:
|
||||
// Negative zero is not not-equal to zero.
|
||||
if (bound == 0) {
|
||||
|
||||
@@ -1903,7 +1903,7 @@ Decoder::DecodeSpecialCondition(Instruction* instr)
|
||||
}
|
||||
break;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case 0xB:
|
||||
if ((instr->Bits(22, 20) == 5) && (instr->Bits(15, 12) == 0xf)) {
|
||||
int Rn = instr->Bits(19, 16);
|
||||
|
||||
+1
-1
@@ -146,7 +146,7 @@ js_dtostr(DtoaState* state, char* buffer, size_t bufferSize, JSDToStrMode mode,
|
||||
case DTOSTR_EXPONENTIAL:
|
||||
MOZ_ASSERT(precision > 0);
|
||||
minNDigits = precision;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case DTOSTR_STANDARD_EXPONENTIAL:
|
||||
exponentialNotation = true;
|
||||
break;
|
||||
|
||||
+6
-6
@@ -5418,7 +5418,7 @@ GCRuntime::incrementalCollectSlice(SliceBudget& budget, JS::gcreason::Reason rea
|
||||
|
||||
incrementalState = State::MarkRoots;
|
||||
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
case State::MarkRoots:
|
||||
if (!beginMarkPhase(reason, lock)) {
|
||||
@@ -5428,7 +5428,7 @@ GCRuntime::incrementalCollectSlice(SliceBudget& budget, JS::gcreason::Reason rea
|
||||
|
||||
incrementalState = State::Mark;
|
||||
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
case State::Mark:
|
||||
AutoGCRooter::traceAllWrappers(&marker);
|
||||
@@ -5472,7 +5472,7 @@ GCRuntime::incrementalCollectSlice(SliceBudget& budget, JS::gcreason::Reason rea
|
||||
if (budget.isOverBudget())
|
||||
break;
|
||||
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
case State::Sweep:
|
||||
if (performSweepActions(budget, lock) == NotFinished)
|
||||
@@ -5482,7 +5482,7 @@ GCRuntime::incrementalCollectSlice(SliceBudget& budget, JS::gcreason::Reason rea
|
||||
|
||||
incrementalState = State::Finalize;
|
||||
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
case State::Finalize:
|
||||
{
|
||||
@@ -5516,7 +5516,7 @@ GCRuntime::incrementalCollectSlice(SliceBudget& budget, JS::gcreason::Reason rea
|
||||
if (isCompacting && isIncremental)
|
||||
break;
|
||||
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
case State::Compact:
|
||||
if (isCompacting) {
|
||||
@@ -5532,7 +5532,7 @@ GCRuntime::incrementalCollectSlice(SliceBudget& budget, JS::gcreason::Reason rea
|
||||
startDecommit();
|
||||
incrementalState = State::Decommit;
|
||||
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
case State::Decommit:
|
||||
{
|
||||
|
||||
+7
-7
@@ -523,13 +523,13 @@ Compare(T* a, T* b, size_t c)
|
||||
{
|
||||
size_t n = (c + size_t(7)) / size_t(8);
|
||||
switch (c % 8) {
|
||||
case 0: do { if (*a++ != *b++) return false; MOZ_FALLTHROUGH;
|
||||
case 7: if (*a++ != *b++) return false; MOZ_FALLTHROUGH;
|
||||
case 6: if (*a++ != *b++) return false; MOZ_FALLTHROUGH;
|
||||
case 5: if (*a++ != *b++) return false; MOZ_FALLTHROUGH;
|
||||
case 4: if (*a++ != *b++) return false; MOZ_FALLTHROUGH;
|
||||
case 3: if (*a++ != *b++) return false; MOZ_FALLTHROUGH;
|
||||
case 2: if (*a++ != *b++) return false; MOZ_FALLTHROUGH;
|
||||
case 0: do { if (*a++ != *b++) return false; [[fallthrough]];
|
||||
case 7: if (*a++ != *b++) return false; [[fallthrough]];
|
||||
case 6: if (*a++ != *b++) return false; [[fallthrough]];
|
||||
case 5: if (*a++ != *b++) return false; [[fallthrough]];
|
||||
case 4: if (*a++ != *b++) return false; [[fallthrough]];
|
||||
case 3: if (*a++ != *b++) return false; [[fallthrough]];
|
||||
case 2: if (*a++ != *b++) return false; [[fallthrough]];
|
||||
case 1: if (*a++ != *b++) return false;
|
||||
} while (--n > 0);
|
||||
}
|
||||
|
||||
+7
-7
@@ -1613,13 +1613,13 @@ FirstCharMatcherUnrolled(const TextChar* text, uint32_t n, const PatChar pat)
|
||||
const TextChar* t = text;
|
||||
|
||||
switch ((textend - t) & 7) {
|
||||
case 0: if (*t++ == pat) return t - 1; MOZ_FALLTHROUGH;
|
||||
case 7: if (*t++ == pat) return t - 1; MOZ_FALLTHROUGH;
|
||||
case 6: if (*t++ == pat) return t - 1; MOZ_FALLTHROUGH;
|
||||
case 5: if (*t++ == pat) return t - 1; MOZ_FALLTHROUGH;
|
||||
case 4: if (*t++ == pat) return t - 1; MOZ_FALLTHROUGH;
|
||||
case 3: if (*t++ == pat) return t - 1; MOZ_FALLTHROUGH;
|
||||
case 2: if (*t++ == pat) return t - 1; MOZ_FALLTHROUGH;
|
||||
case 0: if (*t++ == pat) return t - 1; [[fallthrough]];
|
||||
case 7: if (*t++ == pat) return t - 1; [[fallthrough]];
|
||||
case 6: if (*t++ == pat) return t - 1; [[fallthrough]];
|
||||
case 5: if (*t++ == pat) return t - 1; [[fallthrough]];
|
||||
case 4: if (*t++ == pat) return t - 1; [[fallthrough]];
|
||||
case 3: if (*t++ == pat) return t - 1; [[fallthrough]];
|
||||
case 2: if (*t++ == pat) return t - 1; [[fallthrough]];
|
||||
case 1: if (*t++ == pat) return t - 1;
|
||||
}
|
||||
while (textend != t) {
|
||||
|
||||
+1
-1
@@ -1548,7 +1548,7 @@ ConvertTranscodeResultToJSException(JSContext* cx, JS::TranscodeResult rv)
|
||||
return true;
|
||||
|
||||
default:
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case JS::TranscodeResult_Failure:
|
||||
MOZ_ASSERT(!cx->isExceptionPending());
|
||||
JS_ReportErrorASCII(cx, "generic warning");
|
||||
|
||||
@@ -1513,7 +1513,7 @@ ParseRegExpFlags(const CharT* chars, size_t length, RegExpFlag* flagsOut, char16
|
||||
return false;
|
||||
break;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
default:
|
||||
return false;
|
||||
}
|
||||
|
||||
@@ -3301,7 +3301,7 @@ class BaseCompiler
|
||||
case Scalar::Uint16:
|
||||
case Scalar::Uint32: {
|
||||
isSigned = false;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case Scalar::Int8:
|
||||
case Scalar::Int16:
|
||||
case Scalar::Int32:
|
||||
@@ -3380,15 +3380,15 @@ class BaseCompiler
|
||||
|
||||
switch (access.type()) {
|
||||
case Scalar::Uint8:
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case Scalar::Uint16:
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case Scalar::Int8:
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case Scalar::Int16:
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case Scalar::Int32:
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case Scalar::Uint32: {
|
||||
Register rt = src.tag == AnyReg::I64 ? src.i64().reg.low : src.i32().reg;
|
||||
storeI32(access, ptr, rt);
|
||||
|
||||
@@ -824,7 +824,7 @@ WasmTokenStream::next()
|
||||
return nan(begin);
|
||||
if (!IsWasmDigit(*cur_))
|
||||
break;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case '0': case '1': case '2': case '3': case '4':
|
||||
case '5': case '6': case '7': case '8': case '9':
|
||||
return literal(begin);
|
||||
|
||||
@@ -495,7 +495,7 @@ XPCConvert::JSData2Native(void* d, HandleValue s,
|
||||
(**((nsAString**)d)).SetIsVoid(true);
|
||||
return true;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
}
|
||||
case nsXPTType::T_DOMSTRING:
|
||||
{
|
||||
|
||||
@@ -935,7 +935,7 @@ ProcessArgsForCompartment(JSContext* cx, char** argv, int argc)
|
||||
break;
|
||||
case 'S':
|
||||
ContextOptionsRef(cx).toggleWerror();
|
||||
MOZ_FALLTHROUGH; // because -S implies -s
|
||||
[[fallthrough]]; // because -S implies -s
|
||||
case 's':
|
||||
ContextOptionsRef(cx).toggleExtraWarnings();
|
||||
break;
|
||||
|
||||
@@ -213,7 +213,7 @@ TouchManager::PreHandleEvent(WidgetEvent* aEvent,
|
||||
case eTouchEnd:
|
||||
aIsHandlingUserInput = true;
|
||||
// Fall through to touchcancel code
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eTouchCancel: {
|
||||
// Remove the changed touches
|
||||
// need to make sure we only remove touches that are ending here
|
||||
|
||||
@@ -4404,7 +4404,7 @@ nsCSSRendering::DrawTableBorderSegment(DrawTarget& aDrawTarget,
|
||||
break;
|
||||
case NS_STYLE_BORDER_STYLE_GROOVE:
|
||||
ridgeGroove = NS_STYLE_BORDER_STYLE_GROOVE; // and fall through to ridge
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case NS_STYLE_BORDER_STYLE_RIDGE:
|
||||
if ((horizontal && (twipsPerPixel >= aBorder.height)) ||
|
||||
(!horizontal && (twipsPerPixel >= aBorder.width))) {
|
||||
@@ -4571,7 +4571,7 @@ nsCSSRendering::DrawTableBorderSegment(DrawTarget& aDrawTarget,
|
||||
break;
|
||||
}
|
||||
// else fall through to solid
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case NS_STYLE_BORDER_STYLE_SOLID:
|
||||
DrawSolidBorderSegment(aDrawTarget, aBorder, aBorderColor,
|
||||
aAppUnitsPerDevPixel, twipsPerPixel, aStartBevelSide,
|
||||
|
||||
@@ -1235,7 +1235,7 @@ MakeBorderColor(nscolor aColor, nscolor aBackgroundColor,
|
||||
|
||||
case BorderColorStyleLight:
|
||||
k = 1;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case BorderColorStyleDark:
|
||||
NS_GetSpecial3DColors(colors, aBackgroundColor, aColor);
|
||||
return Color::FromABGR(colors[k]);
|
||||
|
||||
@@ -133,7 +133,7 @@ CSSAlignUtils::AlignJustifySelf(uint8_t aAlignment, LogicalAxis aAxis,
|
||||
}
|
||||
break;
|
||||
case NS_STYLE_ALIGN_STRETCH:
|
||||
MOZ_FALLTHROUGH; // ComputeSize() deals with it
|
||||
[[fallthrough]]; // ComputeSize() deals with it
|
||||
case NS_STYLE_ALIGN_START:
|
||||
offset = marginStart;
|
||||
break;
|
||||
|
||||
@@ -3012,7 +3012,7 @@ MainAxisPositionTracker::
|
||||
case NS_STYLE_JUSTIFY_BASELINE:
|
||||
case NS_STYLE_JUSTIFY_LAST_BASELINE:
|
||||
NS_WARNING("NYI: justify-content:left/right/baseline/last baseline");
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case NS_STYLE_JUSTIFY_FLEX_START:
|
||||
// All packing space should go at the end --> nothing to do here.
|
||||
break;
|
||||
@@ -3220,7 +3220,7 @@ CrossAxisPositionTracker::
|
||||
case NS_STYLE_ALIGN_BASELINE:
|
||||
case NS_STYLE_ALIGN_LAST_BASELINE:
|
||||
NS_WARNING("NYI: align-items/align-self:left/right/self-start/self-end/baseline/last baseline");
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case NS_STYLE_ALIGN_FLEX_START:
|
||||
// All packing space should go at the end --> nothing to do here.
|
||||
break;
|
||||
@@ -3522,7 +3522,7 @@ SingleLineCrossAxisPositionTracker::
|
||||
case NS_STYLE_ALIGN_SELF_START:
|
||||
case NS_STYLE_ALIGN_SELF_END:
|
||||
NS_WARNING("NYI: align-items/align-self:left/right/self-start/self-end");
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case NS_STYLE_ALIGN_FLEX_START:
|
||||
// No space to skip over -- we're done.
|
||||
break;
|
||||
|
||||
@@ -7735,7 +7735,7 @@ nsIFrame::PeekOffset(nsPeekOffsetStruct* aPos)
|
||||
aPos->mWordMovementType = eEndWord;
|
||||
}
|
||||
// Intentionally fall through the eSelectWord case.
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eSelectWord:
|
||||
{
|
||||
// wordSelectEatSpace means "are we looking for a boundary between whitespace
|
||||
|
||||
@@ -4938,7 +4938,7 @@ nsGridContainerFrame::Tracks::AlignJustifyContent(
|
||||
case NS_STYLE_ALIGN_BASELINE:
|
||||
case NS_STYLE_ALIGN_LAST_BASELINE:
|
||||
NS_WARNING("NYI: 'first/last baseline' (bug 1151204)"); // XXX
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case NS_STYLE_ALIGN_START:
|
||||
distribute = false;
|
||||
break;
|
||||
|
||||
@@ -3009,7 +3009,7 @@ nsLineLayout::ExpandRubyBox(PerFrameData* aFrame, nscoord aReservedISize,
|
||||
}
|
||||
// If there are no justification opportunities for space-between,
|
||||
// fall-through to center per spec.
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
}
|
||||
case NS_STYLE_RUBY_ALIGN_CENTER:
|
||||
// Indent all children by half of the reserved inline size.
|
||||
@@ -3201,7 +3201,7 @@ nsLineLayout::TextAlignLine(nsLineBox* aLine,
|
||||
}
|
||||
// Fall through to the default case if we could not justify to fill
|
||||
// the space.
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
}
|
||||
|
||||
case NS_STYLE_TEXT_ALIGN_START:
|
||||
|
||||
@@ -195,7 +195,7 @@ nsPrintPreviewListener::HandleEvent(nsIDOMEvent* aEvent)
|
||||
}
|
||||
}
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eEventAction_Suppress:
|
||||
aEvent->StopPropagation();
|
||||
aEvent->PreventDefault();
|
||||
|
||||
@@ -125,7 +125,7 @@ CSSLexer::NextToken(Nullable<CSSToken>& aResult)
|
||||
|
||||
case eCSSToken_Dimension:
|
||||
resultToken.mText.Construct(token.mIdent);
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eCSSToken_Number:
|
||||
case eCSSToken_Percentage:
|
||||
resultToken.mNumber.Construct(token.mNumber);
|
||||
|
||||
@@ -770,7 +770,7 @@ Declaration::GetPropertyValueInternal(
|
||||
}
|
||||
// tweak aProperty and fall through
|
||||
aProperty = eCSSProperty_border_top;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
}
|
||||
case eCSSProperty_border_top:
|
||||
case eCSSProperty_border_right:
|
||||
@@ -1321,7 +1321,7 @@ Declaration::GetPropertyValueInternal(
|
||||
return;
|
||||
}
|
||||
// Fall through to eCSSProperty_grid_template (syntax #1)
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
}
|
||||
case eCSSProperty_grid_template: {
|
||||
const nsCSSValue& areasValue =
|
||||
@@ -1466,7 +1466,7 @@ Declaration::GetPropertyValueInternal(
|
||||
!justify || !IsSingleValue(*justify)) {
|
||||
return; // Not serializable, bail.
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
}
|
||||
case eCSSProperty_gap: {
|
||||
const nsCSSPropertyID* subprops =
|
||||
|
||||
@@ -164,7 +164,7 @@ AppendFunction(nsCSSKeyword aTransformFunction)
|
||||
break;
|
||||
default:
|
||||
NS_ERROR("must be a transform function");
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eCSSKeyword_translatex:
|
||||
case eCSSKeyword_translatey:
|
||||
case eCSSKeyword_translatez:
|
||||
@@ -2005,7 +2005,7 @@ AddWeightedFilterFunctionImpl(double aCoeff1, const nsCSSValueList* aList1,
|
||||
case eCSSKeyword_invert:
|
||||
case eCSSKeyword_sepia:
|
||||
initialVal = 0.0f;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eCSSKeyword_brightness:
|
||||
case eCSSKeyword_contrast:
|
||||
case eCSSKeyword_opacity:
|
||||
@@ -2255,7 +2255,7 @@ AddShapeFunction(nsCSSPropertyID aProperty,
|
||||
resultFuncArgs->Item(2))) {
|
||||
return nullptr;
|
||||
}
|
||||
MOZ_FALLTHROUGH; // to handle rx and center point
|
||||
[[fallthrough]]; // to handle rx and center point
|
||||
case eCSSKeyword_circle: {
|
||||
// Add circles' |r| (or ellipses' |rx|) values:
|
||||
if (!AddCSSValuePixelPercentCalc(aRestriction == Restrictions::Enable
|
||||
@@ -2471,7 +2471,7 @@ AddTransformLists(double aCoeff1, const nsCSSValueList* aList1,
|
||||
arr->Item(4));
|
||||
break;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
}
|
||||
case eCSSKeyword_matrix:
|
||||
case eCSSKeyword_matrix3d:
|
||||
@@ -2496,7 +2496,7 @@ AddTransformLists(double aCoeff1, const nsCSSValueList* aList1,
|
||||
}
|
||||
break;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eCSSKeyword_interpolatematrix: {
|
||||
// FIXME: If the matrix contains only numbers then we could decompose
|
||||
// here.
|
||||
|
||||
@@ -4580,7 +4580,7 @@ CSSParserImpl::ParseKeyframeSelectorList(InfallibleTArray<float>& aSelectorList)
|
||||
value = 1.0f;
|
||||
break;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
default:
|
||||
UngetToken();
|
||||
// The first time through the loop, this means we got an empty
|
||||
@@ -10759,7 +10759,7 @@ CSSParserImpl::IsLegacyGradientLine(const nsCSSTokenType& aType,
|
||||
haveGradientLine = true;
|
||||
break;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eCSSToken_ID:
|
||||
case eCSSToken_Hash:
|
||||
// this is a color
|
||||
@@ -11442,10 +11442,10 @@ CSSParserImpl::ParseBoxProperties(const nsCSSPropertyID aPropIDs[])
|
||||
switch (count) {
|
||||
case 1: // Make right == top
|
||||
result.mRight = result.mTop;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case 2: // Make bottom == top
|
||||
result.mBottom = result.mTop;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case 3: // Make left == right
|
||||
result.mLeft = result.mRight;
|
||||
}
|
||||
@@ -11488,10 +11488,10 @@ CSSParserImpl::ParseGroupedBoxProperty(int32_t aVariantMask,
|
||||
switch (count) {
|
||||
case 1: // Make right == top
|
||||
result.mRight = result.mTop;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case 2: // Make bottom == top
|
||||
result.mBottom = result.mTop;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case 3: // Make left == right
|
||||
result.mLeft = result.mRight;
|
||||
}
|
||||
@@ -11589,10 +11589,10 @@ CSSParserImpl::ParseBoxCornerRadiiInternals(nsCSSValue array[])
|
||||
switch (countX) {
|
||||
case 1: // Make top-right same as top-left
|
||||
dimenX.mRight = dimenX.mTop;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case 2: // Make bottom-right same as top-left
|
||||
dimenX.mBottom = dimenX.mTop;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case 3: // Make bottom-left same as top-right
|
||||
dimenX.mLeft = dimenX.mRight;
|
||||
}
|
||||
@@ -11600,10 +11600,10 @@ CSSParserImpl::ParseBoxCornerRadiiInternals(nsCSSValue array[])
|
||||
switch (countY) {
|
||||
case 1: // Make top-right same as top-left
|
||||
dimenY.mRight = dimenY.mTop;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case 2: // Make bottom-right same as top-left
|
||||
dimenY.mBottom = dimenY.mTop;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case 3: // Make bottom-left same as top-right
|
||||
dimenY.mLeft = dimenY.mRight;
|
||||
}
|
||||
|
||||
@@ -4109,7 +4109,7 @@ nsComputedDOMStyle::DoGetTextSizeAdjust()
|
||||
switch (StyleText()->mTextSizeAdjust) {
|
||||
default:
|
||||
NS_NOTREACHED("unexpected value");
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case NS_STYLE_TEXT_SIZE_ADJUST_AUTO:
|
||||
val->SetIdent(eCSSKeyword_auto);
|
||||
break;
|
||||
|
||||
@@ -240,7 +240,7 @@ nsRuleNode::EnsureBlockDisplay(StyleDisplay& display,
|
||||
display = StyleDisplay::Block;
|
||||
break;
|
||||
} // else, fall through to share the 'break' for non-changing display vals
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case StyleDisplay::None:
|
||||
case StyleDisplay::Contents:
|
||||
// never change display:none or display:contents *ever*
|
||||
|
||||
@@ -1287,7 +1287,7 @@ nsStyleSVGPaint::Reset()
|
||||
case eStyleSVGPaintType_Server:
|
||||
mPaint.mPaintServer->Release();
|
||||
mPaint.mPaintServer = nullptr;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eStyleSVGPaintType_ContextFill:
|
||||
case eStyleSVGPaintType_ContextStroke:
|
||||
mFallbackColor = NS_RGB(0, 0, 0);
|
||||
|
||||
@@ -639,7 +639,7 @@ MatrixForTransformFunction(Matrix4x4& aMatrix,
|
||||
break;
|
||||
case eCSSKeyword_rotatez:
|
||||
*aContains3dTransform = true;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eCSSKeyword_rotate:
|
||||
ProcessRotateZ(aMatrix, aData);
|
||||
break;
|
||||
|
||||
@@ -108,7 +108,7 @@ SpanningCellSorter::GetNext(int32_t *aColSpan)
|
||||
/* prepare to enumerate the array */
|
||||
mState = ENUMERATING_ARRAY;
|
||||
mEnumerationIndex = 0;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case ENUMERATING_ARRAY:
|
||||
while (mEnumerationIndex < ARRAY_SIZE && !mArray[mEnumerationIndex])
|
||||
++mEnumerationIndex;
|
||||
@@ -138,7 +138,7 @@ SpanningCellSorter::GetNext(int32_t *aColSpan)
|
||||
SortArray, nullptr);
|
||||
mSortedHashTable = sh;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case ENUMERATING_HASH:
|
||||
if (mEnumerationIndex < mHashTable.EntryCount()) {
|
||||
Item *result = mSortedHashTable[mEnumerationIndex]->mItems;
|
||||
@@ -153,7 +153,7 @@ SpanningCellSorter::GetNext(int32_t *aColSpan)
|
||||
return result;
|
||||
}
|
||||
mState = DONE;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case DONE:
|
||||
;
|
||||
}
|
||||
|
||||
@@ -913,7 +913,7 @@ nsTableCellMap::ResetBStartStart(LogicalSide aSide,
|
||||
switch(aSide) {
|
||||
case eLogicalSideBEnd:
|
||||
aRowIndex++;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eLogicalSideBStart:
|
||||
cellData = (BCCellData*)aCellMap.GetDataAt(aRowIndex, aColIndex);
|
||||
if (cellData) {
|
||||
@@ -936,7 +936,7 @@ nsTableCellMap::ResetBStartStart(LogicalSide aSide,
|
||||
break;
|
||||
case eLogicalSideIEnd:
|
||||
aColIndex++;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eLogicalSideIStart:
|
||||
cellData = (BCCellData*)aCellMap.GetDataAt(aRowIndex, aColIndex);
|
||||
if (cellData) {
|
||||
@@ -981,7 +981,7 @@ nsTableCellMap::SetBCBorderEdge(LogicalSide aSide,
|
||||
case eLogicalSideBEnd:
|
||||
rgYPos++;
|
||||
yPos++;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eLogicalSideBStart:
|
||||
lastIndex = xPos + aLength - 1;
|
||||
for (xIndex = xPos; xIndex <= lastIndex; xIndex++) {
|
||||
@@ -1028,7 +1028,7 @@ nsTableCellMap::SetBCBorderEdge(LogicalSide aSide,
|
||||
break;
|
||||
case eLogicalSideIEnd:
|
||||
xPos++;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eLogicalSideIStart:
|
||||
// since bStart, bEnd borders were set, there should already be a cellData entry
|
||||
lastIndex = rgYPos + aLength - 1;
|
||||
|
||||
@@ -6893,7 +6893,7 @@ BCBlockDirSeg::Paint(BCPaintBorderIterator& aIter,
|
||||
if (!aIter.IsTableIEndMost() && (relColIndex > 0)) {
|
||||
col = aIter.mBlockDirInfo[relColIndex - 1].mCol;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eColGroupOwner:
|
||||
if (col) {
|
||||
owner = col->GetParent();
|
||||
@@ -6904,13 +6904,13 @@ BCBlockDirSeg::Paint(BCPaintBorderIterator& aIter,
|
||||
if (!aIter.IsTableIEndMost() && (relColIndex > 0)) {
|
||||
col = aIter.mBlockDirInfo[relColIndex - 1].mCol;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eColOwner:
|
||||
owner = col;
|
||||
break;
|
||||
case eAjaRowGroupOwner:
|
||||
NS_ERROR("a neighboring rowgroup can never own a vertical border");
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eRowGroupOwner:
|
||||
NS_ASSERTION(aIter.IsTableIStartMost() || aIter.IsTableIEndMost(),
|
||||
"row group can own border only at table edge");
|
||||
@@ -6918,7 +6918,7 @@ BCBlockDirSeg::Paint(BCPaintBorderIterator& aIter,
|
||||
break;
|
||||
case eAjaRowOwner:
|
||||
NS_ERROR("program error");
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eRowOwner:
|
||||
NS_ASSERTION(aIter.IsTableIStartMost() || aIter.IsTableIEndMost(),
|
||||
"row can own border only at table edge");
|
||||
@@ -6927,7 +6927,7 @@ BCBlockDirSeg::Paint(BCPaintBorderIterator& aIter,
|
||||
case eAjaCellOwner:
|
||||
side = eLogicalSideIEnd;
|
||||
cell = mAjaCell;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eCellOwner:
|
||||
owner = cell;
|
||||
break;
|
||||
@@ -7104,7 +7104,7 @@ BCInlineDirSeg::Paint(BCPaintBorderIterator& aIter, DrawTarget& aDrawTarget)
|
||||
break;
|
||||
case eAjaColGroupOwner:
|
||||
NS_ERROR("neighboring colgroups can never own an inline-dir border");
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eColGroupOwner:
|
||||
NS_ASSERTION(aIter.IsTableBStartMost() || aIter.IsTableBEndMost(),
|
||||
"col group can own border only at the table edge");
|
||||
@@ -7114,7 +7114,7 @@ BCInlineDirSeg::Paint(BCPaintBorderIterator& aIter, DrawTarget& aDrawTarget)
|
||||
break;
|
||||
case eAjaColOwner:
|
||||
NS_ERROR("neighboring column can never own an inline-dir border");
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eColOwner:
|
||||
NS_ASSERTION(aIter.IsTableBStartMost() || aIter.IsTableBEndMost(),
|
||||
"col can own border only at the table edge");
|
||||
@@ -7123,14 +7123,14 @@ BCInlineDirSeg::Paint(BCPaintBorderIterator& aIter, DrawTarget& aDrawTarget)
|
||||
case eAjaRowGroupOwner:
|
||||
side = eLogicalSideBEnd;
|
||||
rg = (aIter.IsTableBEndMost()) ? aIter.mRg : aIter.mPrevRg;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eRowGroupOwner:
|
||||
owner = rg;
|
||||
break;
|
||||
case eAjaRowOwner:
|
||||
side = eLogicalSideBEnd;
|
||||
row = (aIter.IsTableBEndMost()) ? aIter.mRow : aIter.mPrevRow;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eRowOwner:
|
||||
owner = row;
|
||||
break;
|
||||
@@ -7139,7 +7139,7 @@ BCInlineDirSeg::Paint(BCPaintBorderIterator& aIter, DrawTarget& aDrawTarget)
|
||||
// if this is null due to the damage area origin-y > 0, then the border
|
||||
// won't show up anyway
|
||||
cell = mAjaCell;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case eCellOwner:
|
||||
owner = cell;
|
||||
break;
|
||||
|
||||
@@ -653,7 +653,7 @@ nsTableRowFrame::CalculateCellActualBSize(nsTableCellFrame* aCellFrame,
|
||||
break;
|
||||
}
|
||||
// Fall through to the coord case
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
}
|
||||
case eStyleUnit_Coord: {
|
||||
// In quirks mode, table cell isize should be content-box, but bsize
|
||||
|
||||
@@ -2336,7 +2336,7 @@ nsXULPopupManager::HandleKeyboardEventWithKeyCode(
|
||||
Rollup(0, false, nullptr, nullptr);
|
||||
break;
|
||||
}
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
#endif
|
||||
|
||||
case nsIDOMKeyEvent::DOM_VK_LEFT:
|
||||
@@ -2380,7 +2380,7 @@ nsXULPopupManager::HandleKeyboardEventWithKeyCode(
|
||||
break;
|
||||
}
|
||||
// Intentional fall-through to RETURN case
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
|
||||
case nsIDOMKeyEvent::DOM_VK_RETURN: {
|
||||
// If there is a popup open, check if the current item needs to be opened.
|
||||
|
||||
@@ -645,7 +645,7 @@ NS_IMETHODIMP nsLDAPConnectionRunnable::Run()
|
||||
case LDAP_RES_SEARCH_REFERENCE:
|
||||
// XXX what should we do with LDAP_RES_SEARCH_EXTENDED
|
||||
operationFinished = false;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
default:
|
||||
{
|
||||
msg = new nsLDAPMessage;
|
||||
|
||||
@@ -609,7 +609,7 @@ nsresult nsMsgFilterAfterTheFact::ApplyFilter()
|
||||
// would not have run if move succeeded.
|
||||
m_nextAction = numActions;
|
||||
// Fall through to the copy case.
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case nsMsgFilterAction::CopyToFolder:
|
||||
{
|
||||
nsCString uri;
|
||||
|
||||
@@ -334,7 +334,7 @@ nsresult nsMsgSearchAdapter::EncodeImapTerm (nsIMsgSearchTerm *term, bool really
|
||||
case nsMsgSearchAttrib::ToOrCC:
|
||||
orHeaderMnemonic = m_kImapCC;
|
||||
// fall through to case nsMsgSearchAttrib::To:
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case nsMsgSearchAttrib::To:
|
||||
whichMnemonic = m_kImapTo;
|
||||
break;
|
||||
|
||||
@@ -6450,7 +6450,7 @@ NS_IMETHODIMP nsMsgDBView::NoteChange(nsMsgViewIndex firstLineChanged,
|
||||
// RowCountChanged() will call our GetRowCount()
|
||||
mTree->RowCountChanged(firstLineChanged, numChanged);
|
||||
mRemovingRow = false;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case nsMsgViewNotificationCode::all:
|
||||
ClearHdrCache();
|
||||
break;
|
||||
@@ -6730,7 +6730,7 @@ nsresult nsMsgDBView::NavigateFromPos(nsMsgNavigationTypeValue motion, nsMsgView
|
||||
break;
|
||||
case nsMsgNavigationType::firstUnreadMessage:
|
||||
startIndex = nsMsgViewIndex_None; // note fall thru - is this motion ever used?
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case nsMsgNavigationType::nextUnreadMessage:
|
||||
for (curIndex = (startIndex == nsMsgViewIndex_None) ? 0 : startIndex; curIndex <= lastIndex && lastIndex != nsMsgViewIndex_None; curIndex++) {
|
||||
uint32_t flags = m_flags[curIndex];
|
||||
|
||||
@@ -227,7 +227,7 @@ nsresult nsMsgGroupView::HashHdr(nsIMsgDBHdr *msgHdr, nsString& aHashKey)
|
||||
break;
|
||||
case nsMsgViewSortType::byReceived:
|
||||
rcvDate = true;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case nsMsgViewSortType::byDate:
|
||||
{
|
||||
uint32_t ageBucket;
|
||||
@@ -799,7 +799,7 @@ NS_IMETHODIMP nsMsgGroupView::CellTextForColumn(int32_t aRow,
|
||||
{
|
||||
case nsMsgViewSortType::byReceived:
|
||||
rcvDate = true;
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case nsMsgViewSortType::byDate:
|
||||
{
|
||||
uint32_t ageBucket = 0;
|
||||
|
||||
@@ -174,7 +174,7 @@ static nsresult StripOutGroupNames(char * addresses)
|
||||
group = false;
|
||||
//end of the group, act like a recipient separator now...
|
||||
/* NO BREAK */
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case ',':
|
||||
if (!quoted)
|
||||
{
|
||||
|
||||
@@ -920,7 +920,7 @@ void nsSmtpProtocol::InitPrefAuthMethods(int32_t authMethodPrefValue)
|
||||
MOZ_LOG(SMTPLogModule, mozilla::LogLevel::Error,
|
||||
("SMTP: bad pref authMethod = %d\n", authMethodPrefValue));
|
||||
// fall to any
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case nsMsgAuthMethod::anything:
|
||||
m_prefAuthMethods =
|
||||
SMTP_AUTH_LOGIN_ENABLED | SMTP_AUTH_PLAIN_ENABLED |
|
||||
|
||||
@@ -769,7 +769,7 @@ const char *nsMsgHdr::GetNextReference(const char *startNextRef,
|
||||
// intentional fallthrough so whitespaceEndedAt will definitely have
|
||||
// a non-NULL value, just in case the message-id is not valid (no '>')
|
||||
// and the old-school support is desired.
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
default:
|
||||
if (!whitespaceEndedAt)
|
||||
whitespaceEndedAt = ptr;
|
||||
|
||||
@@ -547,7 +547,7 @@ void Tokenizer::tokenizeHeaders(nsIUTF8StringEnumerator * aHeaderNames, nsIUTF8S
|
||||
if (Substring(headerName, 0, 9).Equals("x-mozilla"))
|
||||
break;
|
||||
// fall through
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case 'u':
|
||||
addTokenForHeader(headerName.get(), headerValue);
|
||||
break;
|
||||
|
||||
@@ -3508,7 +3508,7 @@ NS_IMETHODIMP nsImapMailFolder::ApplyFilterHit(nsIMsgFilter *filter, nsIMsgWindo
|
||||
msgIsNew = false;
|
||||
}
|
||||
// note that delete falls through to move.
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case nsMsgFilterAction::MoveToFolder:
|
||||
{
|
||||
// if moving to a different file, do it.
|
||||
|
||||
@@ -2775,7 +2775,7 @@ void nsImapProtocol::ProcessSelectedStateURL()
|
||||
case nsIImapUrl::nsImapExpungeFolder:
|
||||
Expunge();
|
||||
// note fall through to next cases.
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case nsIImapUrl::nsImapSelectFolder:
|
||||
case nsIImapUrl::nsImapSelectNoopFolder:
|
||||
if (!moreHeadersToDownload)
|
||||
@@ -5624,7 +5624,7 @@ void nsImapProtocol::InitPrefAuthMethods(int32_t authMethodPrefValue)
|
||||
MOZ_LOG(IMAP, LogLevel::Error,
|
||||
("IMAP: bad pref authMethod = %d\n", authMethodPrefValue));
|
||||
// fall to any
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case nsMsgAuthMethod::anything:
|
||||
m_prefAuthMethods = kHasAuthOldLoginCapability |
|
||||
kHasAuthLoginCapability | kHasAuthPlainCapability |
|
||||
|
||||
@@ -2026,7 +2026,7 @@ NS_IMETHODIMP nsParseNewMailState::ApplyFilterHit(nsIMsgFilter *filter, nsIMsgWi
|
||||
}
|
||||
|
||||
// FALLTHROUGH
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case nsMsgFilterAction::MoveToFolder:
|
||||
// if moving to a different file, do it.
|
||||
if (actionTargetFolderUri.get() && !m_inboxUri.Equals(actionTargetFolderUri,
|
||||
|
||||
@@ -1639,7 +1639,7 @@ void nsPop3Protocol::InitPrefAuthMethods(int32_t authMethodPrefValue)
|
||||
MOZ_LOG(POP3LOGMODULE, LogLevel::Error,
|
||||
(POP3LOG("POP: bad pref authMethod = %d\n"), authMethodPrefValue));
|
||||
// fall to any
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case nsMsgAuthMethod::anything:
|
||||
m_prefAuthMethods = POP3_HAS_AUTH_USER |
|
||||
POP3_HAS_AUTH_LOGIN | POP3_HAS_AUTH_PLAIN |
|
||||
@@ -1802,7 +1802,7 @@ int32_t nsPop3Protocol::ProcessAuth()
|
||||
break;
|
||||
case POP3_HAS_AUTH_CRAM_MD5:
|
||||
MOZ_LOG(POP3LOGMODULE, LogLevel::Debug, (POP3LOG("POP CRAM")));
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case POP3_HAS_AUTH_PLAIN:
|
||||
case POP3_HAS_AUTH_USER:
|
||||
MOZ_LOG(POP3LOGMODULE, LogLevel::Debug, (POP3LOG("POP username")));
|
||||
|
||||
@@ -256,7 +256,7 @@ MimeMultipartSigned_parse_line (const char *line, int32_t length, MimeObject *ob
|
||||
mult->hdrs = 0;
|
||||
|
||||
/* fall through. */
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case MimeMultipartSignedBodyFirstHeader:
|
||||
case MimeMultipartSignedBodyHeaders:
|
||||
case MimeMultipartSignedBodyLine:
|
||||
@@ -406,7 +406,7 @@ MimeMultipartSigned_parse_line (const char *line, int32_t length, MimeObject *ob
|
||||
}
|
||||
|
||||
/* fall through. */
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case MimeMultipartSignedSignatureLine:
|
||||
if (hash_line_p)
|
||||
{
|
||||
@@ -497,7 +497,7 @@ MimeMultipartSigned_parse_child_line (MimeObject *obj,
|
||||
case MimeMultipartSignedBodyHeaders:
|
||||
// How'd we get here? Oh well, fall through.
|
||||
NS_ERROR("wrong state in parse child line");
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case MimeMultipartSignedBodyFirstLine:
|
||||
PR_ASSERT(first_line_p);
|
||||
if (!sig->part_buffer)
|
||||
@@ -507,7 +507,7 @@ MimeMultipartSigned_parse_child_line (MimeObject *obj,
|
||||
return MIME_OUT_OF_MEMORY;
|
||||
}
|
||||
/* fall through */
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case MimeMultipartSignedBodyLine:
|
||||
{
|
||||
/* This is the first part; we are buffering it, and will emit it all
|
||||
@@ -551,7 +551,7 @@ MimeMultipartSigned_parse_child_line (MimeObject *obj,
|
||||
case MimeMultipartSignedSignatureHeaders:
|
||||
// How'd we get here? Oh well, fall through.
|
||||
NS_ERROR("should have already parse sig hdrs");
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case MimeMultipartSignedSignatureFirstLine:
|
||||
case MimeMultipartSignedSignatureLine:
|
||||
/* Nothing to do here -- hashing of the signature part is handled up
|
||||
|
||||
@@ -4549,12 +4549,12 @@ nsresult nsNNTPProtocol::ProcessProtocolState(nsIURI * url, nsIInputStream * inp
|
||||
FinishMemCacheEntry(false); // cleanup mem cache entry
|
||||
if (m_responseCode != MK_NNTP_RESPONSE_ARTICLE_NOTFOUND && m_responseCode != MK_NNTP_RESPONSE_ARTICLE_NONEXIST)
|
||||
return CloseConnection();
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case NEWS_FREE:
|
||||
// Remember when we last used this connection
|
||||
m_lastActiveTimeStamp = PR_Now();
|
||||
CleanupAfterRunningUrl();
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case NNTP_SUSPENDED:
|
||||
return NS_OK;
|
||||
break;
|
||||
|
||||
@@ -895,7 +895,7 @@ void TransportLayerDtls::Handshake() {
|
||||
MOZ_MTLOG(ML_ERROR, LAYER_INFO << "Malformed DTLS message; ignoring");
|
||||
// If this were TLS (and not DTLS), this would be fatal, but
|
||||
// here we're required to ignore bad messages, so fall through
|
||||
MOZ_FALLTHROUGH;
|
||||
[[fallthrough]];
|
||||
case PR_WOULD_BLOCK_ERROR:
|
||||
MOZ_MTLOG(ML_NOTICE, LAYER_INFO << "Handshake would have blocked");
|
||||
PRIntervalTime timeout;
|
||||
|
||||
+2
-2
@@ -554,7 +554,7 @@ struct AssertionConditionType
|
||||
* default:
|
||||
* // This case wants to assert in debug builds, fall through in release.
|
||||
* MOZ_ASSERT(false); // -Wimplicit-fallthrough warning in release builds!
|
||||
* MOZ_FALLTHROUGH; // but -Wunreachable-code warning in debug builds!
|
||||
* [[fallthrough]]; // but -Wunreachable-code warning in debug builds!
|
||||
* case 5:
|
||||
* return 5;
|
||||
* }
|
||||
@@ -571,7 +571,7 @@ struct AssertionConditionType
|
||||
#ifdef DEBUG
|
||||
# define MOZ_FALLTHROUGH_ASSERT(reason) MOZ_CRASH("MOZ_FALLTHROUGH_ASSERT: " reason)
|
||||
#else
|
||||
# define MOZ_FALLTHROUGH_ASSERT(...) MOZ_FALLTHROUGH
|
||||
# define MOZ_FALLTHROUGH_ASSERT(...) [[fallthrough]]
|
||||
#endif
|
||||
|
||||
/*
|
||||
|
||||
Loaded 100 of 138 files, more files were not shown because too many files have changed in this diff.
Show more
Reference in new issue
Block a user