ssia
<rdar://problem/57913504>
Created attachment 385608 [details] Patch
Committed r253480: <https://trac.webkit.org/changeset/253480>
Reopening to attach new patch.
Created attachment 385668 [details] Patch
Comment on attachment 385668 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=385668&action=review > Source/WebCore/layout/inlineformatting/InlineFormattingContext.cpp:419 > + auto invert = false; Invert what? > Source/WebCore/layout/inlineformatting/InlineFormattingContext.cpp:427 > + if (!(invert ^ isFirstLine)) invert == isFirstLine is easier to understand than bitwise math. Or something like auto shouldIndent = invert != isFirstLine; if (!shouldIndent) return {}; > Source/WebCore/layout/inlineformatting/InlineFormattingContext.cpp:431 > + return geometry().computedTextIndent(root, usedHorizontalValues.constraints).valueOr(0); > + }; > + lineLogicalLeft += computedTextIndent().valueOr(0); You should decide which level (lambda or caller) converts Optionals to 0s.
Created attachment 385685 [details] Patch
(In reply to Antti Koivisto from comment #6) > Comment on attachment 385668 [details] > Patch lol I used the wrong bug. > > View in context: > https://bugs.webkit.org/attachment.cgi?id=385668&action=review > > > Source/WebCore/layout/inlineformatting/InlineFormattingContext.cpp:419 > > + auto invert = false; > > Invert what? > > > Source/WebCore/layout/inlineformatting/InlineFormattingContext.cpp:427 > > + if (!(invert ^ isFirstLine)) > > invert == isFirstLine is easier to understand than bitwise math. Or > something like > > auto shouldIndent = invert != isFirstLine; > if (!shouldIndent) > return {}; make sense. > > > Source/WebCore/layout/inlineformatting/InlineFormattingContext.cpp:431 > > + return geometry().computedTextIndent(root, usedHorizontalValues.constraints).valueOr(0); > > + }; > > + lineLogicalLeft += computedTextIndent().valueOr(0); > > You should decide which level (lambda or caller) converts Optionals to 0s. copy pasta :(