[Webkit-unassigned] [Bug 222376] [css-grid] Replace the use of -1 with WTF::nullopt

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Feb 25 05:31:38 PST 2021


https://bugs.webkit.org/show_bug.cgi?id=222376

--- Comment #2 from Javier Fernandez <jfernandez at igalia.com> ---
Comment on attachment 421441
  --> https://bugs.webkit.org/attachment.cgi?id=421441
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=421441&action=review

I bealive there are other functions using -1 as indefinite; I think this patch should address all of them. 

See GridTrackSizingAlgorithm::estimatedGridAreaBreadthForChild for instance.

> Source/WebCore/ChangeLog:7
> +

We should explain for each change why we need nullopt instead; those functions return an Optional type and we consider nullopt as indefinite, as we did before with the -1 value. 

It's also need to mention why this change doesn't need new tests, or changes on the already existing ones.
In this case, since we don't expect a behavior change, we don't need to provide tests.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20210225/8f284340/attachment.htm>


More information about the webkit-unassigned mailing list