mirror of https://github.com/bsnes-emu/bsnes.git
427bac3011
byuu says:
I reworked the video sizing code. Ended up wasting five fucking hours
fighting GTK. When you call `gtk_widget_set_size_request`, it doesn't
actually happen then. This is kind of a big deal because when I then go
to draw onto the viewport, the actual viewport child window is still the
old size, so the image gets distorted. It recovers in a frame or so with
emulation, but if we were to put a still image on there, it would stay
distorted.
The first thought is, `while(gtk_events_pending())
gtk_main_iteration_do(false);` right after the `set_size_request`. But
nope, it tells you there's no events pending. So then you think, go
deeper, use `XPending()` instead. Same thing, GTK hasn't actually issued
the command to Xlib yet. So then you think, if the widget is realized,
just call a blocking `gtk_main_iteration`. One call does nothing, two
calls results in a deadlock on the second one ... do it before program
startup, and the main window will never appear. Great.
Oh, and it's not just the viewport. It's also the widget container area
of the windows, as well as the window itself, as well as the fullscreen
mode toggle effect. They all do this.
For the latter three, I couldn't find anything that worked, so I just
added 20ms loops of constantly calling `gtk_main_iteration_do(false)`
after each one of those things. The downside here is toggling the status
bar takes 40ms, so you'll see it and it'll feel a tiny bit sluggish.
But I can't have a 20ms wait on each widget resize, that would be
catastrophic to performance on windows with lots of widgets.
I tried hooking configure-event and size-allocate, but they were very
unreliable. So instead I ended up with a loop that waits up to a maximm
of 20ms that inspects the `widget->allocation.(width,height)` values
directly and waits for them to be what we asked for with
`set_size_request`.
There was some extreme ugliness in GTK with calling
`gtk_main_iteration_do` recursively (`hiro::Widget::setGeometry` is
called recursively), so I had to lock it to only happen on the top level
widgets (the child ones should get resized while waiting on the
top-level ones, so it should be fine in practice), and also only run it
on realized widgets.
Even still, I'm getting ~3 timeouts when opening the settings dialog in
higan, but no other windows. But, this is the best I can do for now.
And the reason for all of this pain? Yeah, updated the video code.
So the Emulator::Interface now has this:
struct VideoSize { uint width, height; }; //or requiem for a tuple
auto videoSize() -> VideoSize;
auto videoSize(uint width, uint height, bool arc) -> VideoSize;
The first function, for now, is just returning the literal surface size.
I may remove this ... one thing I want to allow for is cores that send
different texture sizes based on interlace/hires/overscan/etc settings.
The second function is more interesting. Instead of having the UI trying
to figure out sizing, I figure the emulation cores can do a better job
and we can customize it per-core now. So it gets the window's width and
height, and whether the user asked for aspect correction, and then
computes the best width/height ratio possible. For now they're all just
doing multiples of a 1x scale to the UI 2x,3x,4x modes.
We still need a third function, which will probably be what I repurpose
videoSize() for: to return the 'effective' size for pixel shaders, to
then feed into ruby, to then feed into quark, to then feed into our
shaders. Since shaders use normalized coordinates for pixel fetching,
this should work out just fine. The real texture size will be exposed to
quark shaders as well, of course.
Now for the main window ... it's just hard-coded to be 640x480, 960x720,
1280x960 for now. It works nicely for some cores on some modes, not so
much for others. Work in progress I guess.
I also took the opportunity to draw the about dialog box logo on the
main window. Got a bit fancy and used the old spherical gradient and
impose functionality of nall/image on it. Very minor highlight, nothing
garish. Just something nicer than a solid black window.
If you guys want to mess around with sizes, placements, and gradient
styles/colors/shapes ... feel free. If you come up with something nicer,
do share.
That's what led to all the GTK hell ... the logo wasn't drawing right as
you resized the window. But now it is, though I am not at all happy with
the hacking I had to do.
I also had to improve the video update code as a result of this:
- when you unload a game, it blacks out the screen
- if you are not quitting the emulator, it'll draw the logo; if
you are, it won't
- when you load a game, it black out the logo
These options prevent any unsightliness from resizing the viewport with
image data on it already
I need to redraw the logo when toggling fullscreen with no game loaded
as well for Windows, it seems.
|
||
|---|---|---|
| .. | ||
| button.cpp | ||
| button.hpp | ||
| canvas.cpp | ||
| canvas.hpp | ||
| check-button.cpp | ||
| check-button.hpp | ||
| check-label.cpp | ||
| check-label.hpp | ||
| combo-button-item.cpp | ||
| combo-button-item.hpp | ||
| combo-button.cpp | ||
| combo-button.hpp | ||
| combo-edit-item.cpp | ||
| combo-edit-item.hpp | ||
| combo-edit.cpp | ||
| combo-edit.hpp | ||
| console.cpp | ||
| console.hpp | ||
| frame.cpp | ||
| frame.hpp | ||
| hex-edit.cpp | ||
| hex-edit.hpp | ||
| horizontal-scroll-bar.cpp | ||
| horizontal-scroll-bar.hpp | ||
| horizontal-slider.cpp | ||
| horizontal-slider.hpp | ||
| icon-view-item.cpp | ||
| icon-view-item.hpp | ||
| icon-view.cpp | ||
| icon-view.hpp | ||
| label.cpp | ||
| label.hpp | ||
| line-edit.cpp | ||
| line-edit.hpp | ||
| progress-bar.cpp | ||
| progress-bar.hpp | ||
| radio-button.cpp | ||
| radio-button.hpp | ||
| radio-label.cpp | ||
| radio-label.hpp | ||
| source-edit.cpp | ||
| source-edit.hpp | ||
| tab-frame-item.cpp | ||
| tab-frame-item.hpp | ||
| tab-frame.cpp | ||
| tab-frame.hpp | ||
| table-view-cell.cpp | ||
| table-view-cell.hpp | ||
| table-view-column.cpp | ||
| table-view-column.hpp | ||
| table-view-header.cpp | ||
| table-view-header.hpp | ||
| table-view-item.cpp | ||
| table-view-item.hpp | ||
| table-view.cpp | ||
| table-view.hpp | ||
| text-edit.cpp | ||
| text-edit.hpp | ||
| tree-view-item.cpp | ||
| tree-view-item.hpp | ||
| tree-view.cpp | ||
| tree-view.hpp | ||
| vertical-scroll-bar.cpp | ||
| vertical-scroll-bar.hpp | ||
| vertical-slider.cpp | ||
| vertical-slider.hpp | ||
| viewport.cpp | ||
| viewport.hpp | ||
| widget.cpp | ||
| widget.hpp | ||