Opened 6 years ago
Closed 6 years ago
#14295 closed bug (fixed)
Add FBC padding to BControlLook
| Reported by: | pulkomandy | Owned by: | nobody |
|---|---|---|---|
| Priority: | blocker | Milestone: | R1/beta1 |
| Component: | Kits/Interface Kit | Version: | R1/Development |
| Keywords: | Cc: | jscipione | |
| Blocked By: | Blocking: | ||
| Platform: | All |
Description
It's part of libbe, and used by several apps/libs outside of Haiku (WebKit, Qt, anything using libshared, ACE, ...)
We should freeze it's ABI before shipping beta1.
Change History (8)
comment:1 by , 6 years ago
comment:2 by , 6 years ago
We don't need to solve the bikeshed, only complete and merge https://review.haiku-os.org/#/c/haiku/+/351/
comment:3 by , 6 years ago
| Cc: | added |
|---|
CC jscipione: What's your status here? Easy enough to add the FBC padding, but the actual changes to the API depend on you.
I really don't see how they're needed for the beta, so, if they aren't ready, they just won't make it in.
comment:4 by , 6 years ago
Don't hold up beta on me but would be nice to get fbc padding methods before we freeze the API
comment:5 by , 6 years ago
Oh, I just realized -- BControlLook is in BPrivate. So it doesn't seem to make a lot of sense to add FBC padding while it's there, as we'll need to break ABI to remove it from there, and I don't think we should make such an ABI break so close to beta. Or maybe we should?
comment:6 by , 6 years ago
Added a Gerrit changeset for that: https://review.haiku-os.org/c/haiku/+/464
PulkoMandy, if you think it makes sense (I'm still on the fence), please merge immediately. I'll deal with the package builders later.
comment:7 by , 6 years ago
Mh, I don't know if we should move it out of BPrivate just yet. I think BControlLook is not mature and the API would need a complete review and cleanup before we do that. I would prefer that we keep it private for now so that it doesn't get in the way when we add new control looks and try to get them to look right.
FBC would then just be a way to limit ABI breakage when we make small changes, but not a commitment for complete ABI stability yet.
Well, since nothing outside the tree currently *creates* a BControlLook, only *uses* it, adding FBC padding should not require a mass rebuild. We do need to finish the presently-ongoing bikeshed to determine if there are any other changes to be made, though.