Open Bug 1482053 Opened 6 years ago Updated 2 years ago

trunk/editor/libeditor/EditorDOMPoint.h: 2 * pointless asserts

Categories

(Core :: DOM: Editor, defect, P3)

61 Branch
defect

Tracking

()

UNCONFIRMED

People

(Reporter: dcb314, Unassigned)

Details

User Agent: Mozilla/5.0 (X11; Fedora; Linux x86_64; rv:61.0) Gecko/20100101 Firefox/61.0
Build ID: 20180710113641

Steps to reproduce:

trunk/editor/libeditor/EditorDOMPoint.h:449] -> [trunk/editor/libeditor/EditorDOMPoint.h:449]: (style) Opposite expression on both sides of '||'.
[trunk/editor/libeditor/EditorDOMPoint.h:496] -> [trunk/editor/libeditor/EditorDOMPoint.h:496]: (style) Opposite expression on both sides of '||'.




Expected results:

Source code for first one is

   MOZ_ASSERT(!mOffset.isSome() || mOffset.isSome());

Looks pointless to me.
Component: Untriaged → Editor
Priority: -- → P3
Product: Firefox → Core
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.