[Scummvm-devel] R: FORCE_RTL
sunmax at libero.it
sunmax at libero.it
Sun Apr 10 00:02:32 CEST 2011
Hi there Max & Team,
(I hope I'm not double-posting but I didn't see my message coming through the
other mail account)
>> I am not thrilled about this at all: FORCE_RTL always was a hack. In fact,
IIRC it was always meant to be temporary (right, Sunmax? ;)
Eh, eh ;-)
It was a good hack, if some other people liked it, but you are right, still a
hack.
>> Hey, it probably was introduced before we ever added those flags...
Correct.
IIRC it was introduced when there was no official support for that feature,
and lot of requests from PS2 ScummVM enthusiasts, because of the long time to
reboot ScummVM otherwise.
>> This is why some engines are marked as not supporting RTL
Well, on some embedded devices, we should still offer the users the option to
"force RTL" for those too, they might leak few KB, but at least the final user
won't have to wait 5 minutes before starting the next ScummVM adventure, so it
sounds a fair trade to me. I think time is a more precious resource than
memory.
I'm heading to the US Embassy in Rome to renew the first of my 2 VISAs (as if
life wasn't already complicated enough...) next week, when I come back I can
have a look to a cleaner implementation.
But my opinion is that there should still be a way to override those metaflags
(if the users prefer it that way, and I already now for a fact the PS2 ones
do), and always get back to main launcher.
Thanks,
-max
>----Messaggio originale----
>Da: max at quendi.de
>Data: 09/04/2011 23.33
>A: "ScummVM devel"<scummvm-devel at lists.sourceforge.net>
>Ogg: [Scummvm-devel] FORCE_RTL
>
>Hi,
>
>I just saw these commits:
>
>Am 09.04.2011 um 18:44 schrieb dhewg:
>
>> This automated email contains information about 4 new commits which have
been
>> pushed to the 'scummvm' repo located at https://github.com/scummvm/scummvm
.
>>
>> Summary:
>> e2a0c09692 CONFIGURE: Use FORCE_RTL for the Wii backend
>> 44ab256101 CONFIGURE: Use FORCE_RTL for the Android backend
>> 90f47c7b1d ANDOID: Replace mkmanifest.pl with sh/sed code
>> b0351e1b85 ANDROID: Automate the Manifest's versionCode
>
>I am not thrilled about this at all: FORCE_RTL always was a hack. In fact,
IIRC it was always meant to be temporary (right, Sunmax? ;). Seeing its use
proliferate and no sign of anybody thinking about the ramifications is not
ideal :/.
>
>Why is this a hack? Because RTL is *still* not working correctly for many
engines. The primary reason is the incorrect use of global / static variables,
which are not reset when an engine is re-run. And some engines surely still
leak memory. This is why some engines are marked as not supporting RTL
>Now, unless I am mistaken, the FORCE_RTL hack totally ignores those (meta)
engine flags. Hey, it probably was introduced before we ever added those
flags...
>
>So, I am very much tempted of ripping out the relevant code for FORCE_RTL
right now, and thus force people who need this hack to finally work on a proper
solution :). Which might involve coaxing a lot of engine authors into fixing
their engines once and for all (I don't see why Eugene and me should be the
only ones who do that ;).
>
>
>Bye,
>Max
>------------------------------------------------------------------------------
>Xperia(TM) PLAY
>It's a major breakthrough. An authentic gaming
>smartphone on the nation's most reliable network.
>And it wants your games.
>http://p.sf.net/sfu/verizon-sfdev
>_______________________________________________
>Scummvm-devel mailing list
>Scummvm-devel at lists.sourceforge.net
>https://lists.sourceforge.net/lists/listinfo/scummvm-devel
>
More information about the Scummvm-devel
mailing list