|Re: [hatari-devel] Open Hatari in big window|
[ Thread Index |
| More lists.tuxfamily.org/hatari-devel Archives
- To: hatari-devel@xxxxxxxxxxxxxxxxxxx
- Subject: Re: [hatari-devel] Open Hatari in big window
- From: Thomas Huth <th.huth@xxxxxxxxx>
- Date: Thu, 5 Mar 2020 09:51:15 +0100
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/simple; d=posteo.de; s=2017; t=1583398277; bh=v+fOLH9Es435xRIqc5ZN1dbwp3zXg6taFvFgaF+7P7E=; h=Date:From:To:Subject:From; b=FV+1YXw4HoWG41SI6RkBou9jynIdcbk/iq7j/+1SEMSXFWC2yUaDdzYqa9rfl4AaR LD0s0MgBIs5XpetOpQ7q/OaXkZ3xKVAc2payT1NA5CzVF79k+qBbUAAjXzp8Wau/wQ 0r2spg5D7MyQlq0mZZv43CRc0XhMLQGk97HoXDdkiH375ImeR8iApJriD/Ue2IotV/ nJPUJVNc+oA9rbp1ARbpU2oaYzS4rudiUnjySso6M7EHUEhg67z7agfu91ZbnRpmAJ 9o9gkgC/YSXgKMvL3UNkrlFccydBSoMrT5+N8sc/Fm2Iuqah0WxEOIMb6Kmg5AH/am Cub3cVREn3ilQ==
Am Thu, 5 Mar 2020 01:54:32 +0200
schrieb Eero Tamminen <oak@xxxxxxxxxxxxxx>:
> On 3/3/20 8:25 AM, Thomas Huth wrote:
> > Am Mon, 2 Mar 2020 22:58:18 +0200
> > schrieb Eero Tamminen <oak@xxxxxxxxxxxxxx>:
> >> On 3/2/20 4:14 PM, Thomas Huth wrote:
> >>> Looks reasonable to me at a first glance. Do we also want to
> >>> support floating point values for the -z parameter, too? In that
> >>> case, you also have to replace the "atoi()" there.
> >> Attached patch does that, but then I needed to
> >> modify also RenderQuality, as linear scaling will
> >> look horrible with non-integer scaling factors,
> >> whereas that's the only one that doesn't blur
> >> the output when using integer scaling factor.
> > I don't think that modifying RenderQuality automatically is such a
> > good idea here. That will only cause confusion e.g. when the users
> > have a look at their hatari.cfg afterwards and wonder why the
> > setting was changed...
> If it's set automatically to the best value,
> user option could be removed. :-)
I'm not sure whether automatic linear scaling is really always the best
value for everbody ... there might be a perfomance impact for this on
old systems, too?
Maybe we should not care about such old systems anymore, but I think
that's rather something that we should revisit after the next release
has been done, and not do it now before the upcoming release.
> > they should change the RenderQuality setting manually instead.
> In that case, there should be at least command
> line option for it, so that user can set it
> along with the zoom option.
> I'm OK either way.
Thinking about this for a while, I guess the automatic way is likely
better than adding more config options (we've got too many of those
already anyway)... but let's wait for the next release first. I think
after the release, we should do a major clean-up and also remove SDL1
support (SDL1 is decomissioned since more than 6 years now, so
everybody should now really use SDL2 these days).
> Btw. Why there's an option for disabling
> SdlRenderer usage, is it e.g. because it needs
> working GLES support, and all platforms might
> not have that?
Yes, that was the original intention. Not sure whether we still need
this now in 2020 since every recent machine should have GL support
now... maybe another thing to clean up after the next release...