Closed Bug 1523140 Opened 5 years ago Closed 5 years ago

Derive more and shrink repr(c) representation of MaxLength / MozLength.

Categories

(Core :: CSS Parsing and Computation, defect, P2)

defect

Tracking

()

RESOLVED FIXED
mozilla67
Tracking Status
firefox67 --- fixed

People

(Reporter: emilio, Assigned: emilio)

References

Details

Attachments

(1 file)

No description provided.
Depends on: 1523071

This patch:

  • Makes LengthPercentageOrAuto generic, and removes a bunch of code fo
    LengthPercentageOrNone, which was used only for servo and now can use the
    normal MaxLength (with a cfg() guard for the ExtremumLength variant).

  • Shrinks MaxLength / MozLength's repr(C) reperesentation by reducing enum
    nesting. The shrinking is in preparation for using them from C++ too, though
    that'd be a different bug.

  • Moves NonNegative usage to the proper places so that stuff for them can be
    derived.

I did this on top of bug 1523071 to prove both that it could be possible and
that stuff wasn't too messy. It got a bit messy, but just because of a bug I
had fixed in bindgen long time ago already, so this updates bindgen's patch
version to grab a fix instead of ugly workarounds :)

Priority: -- → P2
Pushed by emilio@crisal.io:
https://hg.mozilla.org/integration/autoland/rev/5a801f6c5c8f
Derive more length stuff, and shrink MaxLength / MozLength's repr(C) representation. r=boris
Blocks: 1526736
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: