Closed Bug 1259802 Opened 4 years ago Closed 4 years ago

Add type replacement annotations to simplify rust binding generation for nsStyleStruct.h

Categories

(Core :: CSS Parsing and Computation, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla48
Tracking Status
firefox48 --- fixed

People

(Reporter: emilio, Assigned: emilio)

References

Details

Attachments

(1 file, 1 obsolete file)

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Firefox/45.0
Build ID: 20160319124722

Steps to reproduce:

Patch to generate replacements for some structs which generate complex bindings in nsStyleStruct.h
Attached patch Add ns* structs replacement (obsolete) — Splinter Review
Comment on attachment 8734842 [details] [diff] [review]
Add ns* structs replacement

Review of attachment 8734842 [details] [diff] [review]:
-----------------------------------------------------------------

r=me with that fix.

::: layout/style/nsStyleStruct.h
@@ +102,3 @@
>  
> +/**
> + * If something in this types or the assertions below needs to change, ask

I would add an explanation here that these are used to map Gecko types to simpler layout-equivalent types to aid rust binding generation.
Attachment #8734842 - Flags: review+
Assignee: nobody → ecoal95
Blocks: stylo
Summary: stylo: Required patch for sane rust bindings → Add type replacement annotations to simplify rust binding generation for nsStyleStruct.h
Attachment #8734842 - Attachment is obsolete: true
Attachment #8734849 - Flags: review?(bobbyholley)
(In reply to Bobby Holley (busy) from comment #2)
> Comment on attachment 8734842 [details] [diff] [review]
> Add ns* structs replacement
> 
> Review of attachment 8734842 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> r=me with that fix.
> 
> ::: layout/style/nsStyleStruct.h
> @@ +102,3 @@
> >  
> > +/**
> > + * If something in this types or the assertions below needs to change, ask
> 
> I would add an explanation here that these are used to map Gecko types to
> simpler layout-equivalent types to aid rust binding generation.

Done
Attachment #8734849 - Flags: review?(bobbyholley) → review+
https://hg.mozilla.org/mozilla-central/rev/745c69669283
Status: UNCONFIRMED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in before you can comment on or make changes to this bug.