[Scummvm-devel] Merging the vkeybdy and GUI branches

[Vicent Marti]

Hey there Max, everybody,

regarding your questions on my branch:

- As far as I know, there's no core functionality missing (correct me
if I'm wrong). Regarding the performance issues, Joost has reported
severe slowdowns on the debug menus of a certain engine (can't recall
what it was). Everything else seems to be fine on high-end platforms,
although some more work could be put on fixing bottlenecks for the
slower platforms. The slow loading times in all platforms are, afaik,
fixed.

- I haven't ran any memory benchmarks -- theoretically it should be
taking slightly less memory than the old implementation since we don't
load as many bitmaps on memory (vectorial renderer and all that).
DrMcCoy went very anal with Valgrind logs, so I ran some intense
bugfix sessions until there wasn't a single leak left. That was
however about 2 weeks before the project end. Code written after that
may leak some, although I tried to be much more careful.

- I have found my PDA USB cable while unpacking, expect some reports
from me about the performance of a moderately high-end smartphone
(HTC430 I think), as soon as I manage to build the WinCE port.

- The RTL merge isn't there yet, I can do it tomorrow morning.

I have just finished unpacking my computer rigs on my new flat (hah,
finally), and everything seems to be working fine. Expect some more
performance fixes in the following days.

Cheers.

Vicent Martí
----------------
http://www.smartlikearoboc.com



On Thu, Sep 18, 2008 at 1:35 PM, Max Horn <max at quendi.de> wrote:
> Hi folks,
>
> so, the RTL branch was merged successfully some time ago. There's still
> plenty of polishing required, I think: E.g. we plan to make it possible for
> engines to hook into the GMM (global main menu), so that they can add
> Load/Save/Help buttons (not sure whether we should restrict it to that fixed
> list, or allow arbitrary ones). That way, we will be able to fully merge the
> SCUMM main menu dialog and the GMM. Also, Filippos expressed some
> interested, he wants to use this for Drascula. Anybody who is interested in
> working with the two on this, speak up; also, feel free to ask me, I have
> some ideas about how to tackle this.
>
> And while I am at it, there's a huge FIXME in engines/dialogs.cpp that needs
> to be resolved... ;)
>
>
> But the main purpose of this mail is to get the merge of the vkeybd & GUI
> branches going. We had roughly agreed to first merge vkeybdy then GUI, due
> to interdepencenies between the two branches. However, this may be changed
> if it turns out that GUI is  much closer to branching than vkeybd.
>
> Now, I'd really like to hear from Stephen & Joost, respectively from Vicent
> & Johannes, what the status of the respective branches are. What are in your
> eyes the major blockers for a merge? Please tell us. That way, the rest of
> the team can help actively.
>
> In particular, for the GUI branch:
> * which functionality is missing?* what is known to be slow?
> * how is memory usage compared to the current GUI (looking e.g. at top, or
> better tools)? Did anybody valgrind it yet?
> * everybody out there with "low-end" devices, such as phones, PDAs,
> consoles, is *highly welcome to try out the GUI branch and tell us about it
> * did you perform a merge with trunk after the RTL branch was merged? If
> not, please do one soon; besides RTL, we have major changes in trunk, but
> hopefully, they won't be a problem for this branch
> * ...
>
> For the vkeybdy branch:
> * which functionality is missing?
> * do we have a proper *free* graphics pack yet?
> * did anybody perform testing on a device which might actually benefit from
> it?
> * as with the GUI branch: did you recently perform a merge? If not, can you
> do one, please?
> * ...
>
>
> Cheers,
> Max
>