Skip to content
This repository has been archived by the owner on Feb 10, 2024. It is now read-only.

[SDL2] overwriting of window borders in some conditions on amigaos4 #146

Closed
kas1e opened this issue Jun 1, 2020 · 2 comments
Closed

[SDL2] overwriting of window borders in some conditions on amigaos4 #146

kas1e opened this issue Jun 1, 2020 · 2 comments
Assignees
Labels
bug Something isn't working OS:amigaos4

Comments

@kas1e
Copy link

kas1e commented Jun 1, 2020

As we discuss on amigans there is the same kind of issue as we had with shaderjoy when resizing a window from bigger to smaller size. Quote from Capehill:

Reproducible with some examples: testoverlay2, teststreaming. Not producible with testdrawchessboard.

Originally issue was found with some sdl2 port, which initialization code looks like this:

window_ = SDL_CreateWindow(WINDOW_TITLE,
                           SDL_WINDOWPOS_UNDEFINED, SDL_WINDOWPOS_UNDEFINED,
                           pop_window_width, pop_window_height, flags);
// Make absolutely sure that VSync will be off, to prevent timer issues.
SDL_SetHint(SDL_HINT_RENDER_VSYNC, "0");
renderer_ = SDL_CreateRenderer(window_, -1 , SDL_RENDERER_ACCELERATED | SDL_RENDERER_TARGETTEXTURE);
SDL_RendererInfo renderer_info;
if (SDL_GetRendererInfo(renderer_, &renderer_info) == 0) {
    if (renderer_info.flags & SDL_RENDERER_TARGETTEXTURE) {
        is_renderer_targettexture_supported = true;
    }
}

Which means issues probably happens in compositing renderer path.

@capehill capehill added bug Something isn't working OS:amigaos4 labels Jun 15, 2020
@capehill
Copy link
Collaborator

testdrawchessboard uses SW rendering, so blitting happens via SDL_os4framebuffer.c and there are differences when compared to blitting in SDL_render_compositing.c.

  1. SDL_Window vs. Window width & height. Maybe there are not in sync
  2. Framebuffer code uses min() to limit blitting area

Should be also tested with MiniGL renderer. Ogles2 is not affected because ogles2 handles the blitting internally.

@capehill capehill self-assigned this Jun 16, 2020
capehill added a commit that referenced this issue Jul 14, 2020
Avoid blitting over window borders (#146)
@capehill
Copy link
Collaborator

Should be fixed but if not, please re-open.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working OS:amigaos4
Projects
None yet
Development

No branches or pull requests

2 participants