Re: XPIDL trajectory

2018-06-26 Thread Mike Hommey
On Wed, Jun 27, 2018 at 01:24:27PM +1000, Nicholas Nethercote wrote: > Thanks for the tip. I've updated the last part of the script accordingly: > > # Count number of bytes in the xptdata.o file. > cd o64 > echo -n "xptdata.o bytes: " > wc --bytes xpcom/reflect/xptinfo/xptdata.o > cd .. > > The

Re: XPIDL trajectory

2018-06-26 Thread Nicholas Nethercote
Thanks for the tip. I've updated the last part of the script accordingly: # Count number of bytes in the xptdata.o file. cd o64 echo -n "xptdata.o bytes: " wc --bytes xpcom/reflect/xptinfo/xptdata.o cd .. The current measurements are now: Tue, Jun 26, 2018: m-i 423583:4a20ed6e2fee .idl files:

Re: XPIDL trajectory

2018-06-26 Thread Nika Layzell
Measuring the size of the xpt files is no longer as relevant. Since mccr8 and I rewrote xptinfo, they're only used during the build step, and are not bundled. They're combined into a single .cpp file (xptdata.cpp) which generates shared static pages (mostly) without relocations :-). Perhaps

Re: XPIDL trajectory

2018-06-25 Thread Boris Zbarsky
On 6/26/18 12:45 AM, L. David Baron wrote: What's the relative value of making something not use xpidl anymore vs. marking an xpidl interface as no longer [scriptable]? I think the main value is in not using xpidl anymore, for two reasons: 1) You can't devirtualize things while still using

Re: XPIDL trajectory

2018-06-25 Thread Kris Maglione
On Mon, Jun 25, 2018 at 09:45:22PM -0700, L. David Baron wrote: On Tuesday 2018-06-26 14:29 +1000, Nicholas Nethercote wrote: The trend is clearly down, except for the large increase in .xpt size for the most recent measurement -- note the extra digit! It appears that .xpt files used to be

Re: XPIDL trajectory

2018-06-25 Thread L. David Baron
On Tuesday 2018-06-26 14:29 +1000, Nicholas Nethercote wrote: > The trend is clearly down, except for the large increase in .xpt size for > the most recent measurement -- note the extra digit! It appears that .xpt > files used to be binary, and now they are JSON. This might be related to > mccr8's

XPIDL trajectory

2018-06-25 Thread Nicholas Nethercote
Hi, After Firefox 57 removed support for legacy extensions, I decided to (roughly) track how much XPIDL code we have. Here are some measurements: Fri, Aug 4, 2017: m-i 372493:72873c109b1b .idl files: 1167 .idl lines: 110240 total .xpt bytes: 417621 total Thu, Aug 17, 2017: m-i