The portion of the window that the game canvas covers is now what is
returned by video::input_area(). This is in display coordinates, which
may not match draw coordinates or output coordinates.
sdl/input.cpp was changed to query and account for input_area when the
mouse location is polled. This removes the input dimension caching that
was previously being done.
All other uses of input_area() are probably erroneous, as SDL events are
automatically converted to draw-space by SDL. It is only needed when
polling the mouse location directly, which returns it in display
coordinates relative to the window.
This ensures sveral things:
* the game display's offscreen render buffers are used correctly
* custom render targets will report the correct draw area
* they will also report the correct clipping region and viewport
* converting draw space coordinates to output coordinates will
return the correct coordinates on the full render target.
Not much of this is currently used, but it will be necessary to get
correct for source clipping regions and possibly input conversion.
These are both on the TODO list to clean up.
Theoretically the drawing area may change if we are rendering to
texture. However the vast majority of cases wanted the game canvas
size independent of whatever the current render target may be.
A function game_canvas_size() was also added returning a point,
and this was used to simplify things where appropriate.
* faked() has been renamed headless() to better express its meaning
- it is true when there is no window or renderer at all
* internal usage of the window pointer has been clarified
- a window should always exist, unless in headless mode
* non_interactive() has been removed in favour of headless()
- they are logically identical
* some nomenclature internal to video:: was clarified
- fake_interactive_ -> testing_
- fake_size -> test_resolution_
- init_fake_window() -> init_test_window()
- make_fake() -> init_fake()
- update_framebuffer_fake() -> update_test_framebuffer()
* any_fake() was replaced with testing() to check if running tests
* make_test_fake() was removed in favour of set_resolution()
- the only other use was for init which can be done... in init()
There are now two clear non-standard modes: headless() and testing().
Headless mode has no window or renderer, and hardware rendering
should never be attempted.
Testing mode has a window and renderer, but there are not currently
any tests which use them. Rendering functions may be skipped.
* don't apply a delay while calculating fps count - this is redundant
* remove dirty_ flag - it can be replaced by queueing a redraw
* rename redraw_everything to queue_rerender for clarity
* remove some unnecessary redundant actions when queueing rerenders
* make draw() private and stop everything from calling it
- calling it manually is no longer necessary, and now harmful.
* the dialog now raises a resize event in this case
* raise_resize_event() now activates linked handlers immediately,
in stead of waiting until the next event pump.
* removed clear_screen() - not necessary to do
* removed delay() - inappropriately used, and should not be here anyway
* expose render_screen only to draw_manager - nothing else should do it
* remove read_texture - unused
* split window_has_flags into individual queries:
- window_is_visible()
- window_has_focus()
- window_has_mouse_focus()
* tidied window mode setting code
* removed setup_completed() and has_window() - always true as used
* cleaned video.hpp includes
* deexposed some internal video functions
- update_framebuffer -> update_buffers(false)
- make_fake -> unused externally
The only real change is that in stead of passing around CVideo
pointers and calling things via CVideo::get_singleton().foo()
one simply does not pass around pointers, and calls video::foo().
* removed DRAW_ALL_EVENT - it is not the correct way to draw everything
* made debug_clock a TLD, in stead of hooking DRAW events
* tidied draw event handling, also removing some incorrect usage
* marked many functions for refactor
* removed some unused functions
* moved help tooltip functions out to floating_label.*pp
* streamlined help string tracking
* removed flip_locker - it is no longer useful
* removed update_locker - it is no longer useful
It did not match the actual animation placement... the code was both
duplicated and wrong. I guess one part got updated and the other part
didn't.
It still needs to be deduplicated...
TLDs can now be deregistered or raised during TLD iteration.
Also removed an explicit deregister of the loading screen, which
appears to no longer be necessary.
