Re: When I'm 64 ( nsFrameStateBits.h )

"L. David Baron" <[email protected]>
Newsgroups gmane.comp.mozilla.devel.layout
Message-ID <[email protected]>
On Saturday 2016-02-13 18:13 -0800, Jet Villegas wrote:
> Hi All:
> 
> While hacking on bug 1242461, I ran into the issue where our 64 bits of
> nsFrame and nsBlockFrame state are used up. To resolve this, and I was
> thinking I could reuse bit 59 to indicate NS_FRAME_EXTENDED_BITS and
> allocate another uint32 or uint64 for those bits. Has someone already tried
> this? Will the memory hit be unacceptable?

I don't think it should be separately allocated; it should just be a
fully-fledged part of nsIFrame.

It looks like there's a 32-bit gap on 64-bit builds next to
mOverflow, and we should probably just put another 32 bits of frame
state bits there (after double-checking this).  We might want to
separate the class-specific bits into that new 32-bit value,
although I'm less sure about that.

-David

-- 
π„ž   L. David Baron                         http://dbaron.org/   𝄂
𝄒   Mozilla                          https://www.mozilla.org/   𝄂
             Before I built a wall I'd ask to know
             What I was walling in or walling out,
             And to whom I was like to give offense.
               - Robert Frost, Mending Wall (1914)

_______________________________________________
dev-tech-layout mailing list
[email protected]
https://lists.mozilla.org/listinfo/dev-tech-layout
signature.asc (application/pgp-signature, 819 B)
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iQIcBAEBCAAGBQJWv+SfAAoJEO/hYSUPhPwh7x4P/Rsj+GUrAbMnDP4QW34vFznu
Sb9GRt9T4F56uHoNIviGvsTbCeShDRvkRTytDboN67lpfx9Bh6H/UL4iQ0cAGbKE
4TvmcWVM2ksG1YvqaQFpgY/Qzxn7v+MYppnFfaQn8my/mqGnZGKgq8r3ZJz//ZiF
xX5BZqID+Bu0Le/RbhZF/yoiePnySRr/Pn3iprom15NJfzIz4DqcXa+4LoWr+2pR
g3z2Y929XXJc1HcGx6Ix0nS2OFRXetc8MB152UcINa/RNF/cl8m1eqqyfMmv43A5
4nqwNgq+3dudwUOCA1JpCovSvcUb3i28e4VrQWUzwfxKfcEFrbbZV1ulMO8kBk5A
Mo52xnaXfd0i2O7VoQmK7JWwCfCJLXSUcpcJ2SwvvezeLCd2UjaEDuwGkmkS1WJY
/J6MAOmbxrvdzPna97ZcSzu6cR+hdmC/tr1MgCp99BFfz4ZwdgdE3G3qXZODygyo
nXbz3WV5vdAR4CfW3ZepnlX9Axkj0gYDYJfFnrrc87r/8uzwGYzozFvAQCizP3rW
Rz+AFkuKxrlYv3CnhIsx+E0zXb9I04aTAgW2e5MYzyqdW60np4u514cAJe256kUI
2YASBkJkKJZhXs8fljlrowftmBuF/d6qBQlbYzmFHdtEcAVaJSYlqKHdt8PlwaGE
ZjpyAr0kXIpf/I4u2kep
=9+n5
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.