<html><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>Greetings people, thanks for pointing out all this.</div><div><br></div><div>Begasus: The files that fail to be patched are the ones who should be removed from the build altogether, but that somehow "svn diff" doesn't properly define as deleted in the patch file. These can be safely ignored, they aren't compiled at all if you are building with the Makefile.</div><div><br></div><div>Regarding the lack of the modern theme when launching the port: The theme filenames have changed in the new GUI. Make sure that you have the "scummodern.zip" file somewhere where the executable can access it (e.g. same folder as the exe or in the themes path that you defined on the settings) and then remove your old scummvm.ini or just manually select the modern theme from the options menu. It will load automatically from there on.</div><div><br></div><div>I have just added a small hotfix which will automatically set ScummVM to use the new modern theme if it finds a reference to the old theme on the ini file.</div><div><br></div><div>md5: I have fixed most of your nitpicks. I do believe that the removal of drawing hints in all files is correct, since most of them were superfluous because of default values. I'll look into it more carefully. Unfortunately, I don't have access to a Windows computer right now, and I can't run batch files on my MacBook. Hopefully a kind soul will update the MSVS solutions once the merge is in place.</div><div><br></div><div>Thanks for the input -- although I'm glad that the trunk compiles (at least for Begasus), right now I don't know if I should stick to the plan of doing the merge this afternoon or just delay it and pray I can hack some kind of Wifi at my new place tomorrow or so (I'm going on holidays! Yay for 9 d'Octubre).</div><div><br></div><div>I'll keep you updated.</div><div><br></div><div>Cheers,</div><div>Vicent</div><div><br></div><br><div><div>On Oct 8, 2008, at 1:52 PM, Filippos K wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><span class="Apple-style-span" style="border-collapse: separate; color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0; "><div class="hmmessage" style="font-size: 10pt; font-family: Tahoma; "><div style="text-align: left; ">Hello Marti<br><br>At a first glance, your patch seems ok. I haven't had time to test it, since I'm at work now, but I do have some small comments/nitpicks :)<br>- There's a TANOKU-TODO in gui/dialog.h, which should be changed to a TODO<br>- I think that file gui/themes/scummodern/cursor.bmp is not properly defined as a binary file (the patch contains the file contents as text)<br></div>- scummtheme: perhaps this should mention and check what kinds of file extensions it parses? It's a bit ambiguous right now<br>- I noticed that some drawing hints in the current code have been removed in your patch altogether, without them being replaced by the drawing types you specified (e.g. in gui/console.cpp). Is this correct?<br>- Your patch only updates the MSVC9 project files, you can use the batch files inside the /dists folder to update all of them<br><br>Other than that, it looks ok to me at a first glance, nice work :)<br><br>I'll try and check if it compiles and works for me tonight<br><br>Regards<br>Filippos (md5/thebluegr)<br><br><hr id="stopSpelling">> Date: Wed, 8 Oct 2008 10:37:23 +0200<br>> From:<span class="Apple-converted-space"> </span><a href="mailto:tanoku@gmail.com">tanoku@gmail.com</a><br>> To:<span class="Apple-converted-space"> </span><a href="mailto:max@quendi.de">max@quendi.de</a><br>> CC:<span class="Apple-converted-space"> </span><a href="mailto:scummvm-devel@lists.sourceforge.net">scummvm-devel@lists.sourceforge.net</a><br>> Subject: Re: [Scummvm-devel] Merging the GUI branch<br>><span class="Apple-converted-space"> </span><br>> Hello gentlemen:<br>><span class="Apple-converted-space"> </span><br>> Since nobody has complained about the patch in the last 36h, I'm gonna<br>> assume it *is* working (it seems to work pretty well over here,<br>> anyways), and will commit the GUI merge to the trunk this evening;<br>> that is, unless someone has any last minute objections.<br>><span class="Apple-converted-space"> </span><br>> You'll be able to find me on IRC when the trunk is totally trashed.<br>><span class="Apple-converted-space"> </span><br>> Cheers,<br>><span class="Apple-converted-space"> </span><br>> Vicent Martí<br>> ----------------<br>><span class="Apple-converted-space"> </span><a href="http://www.smartlikearoboc.com">http://www.smartlikearoboc.com</a><br>><span class="Apple-converted-space"> </span><br>><span class="Apple-converted-space"> </span><br>><span class="Apple-converted-space"> </span><br>> On Mon, Oct 6, 2008 at 7:32 PM, Vicent Marti <<a href="mailto:tanoku@gmail.com">tanoku@gmail.com</a>> wrote:<br>> > Ah, the magical wonders of Subversion!! "Version control done right", they<br>> > said.<br>> ><br>> > Ok, on topic, the merge is ready and sitting on my hard disk. I've compiled,<br>> > done playtesting, ran all unit tests (pointless, I didn't even touch the<br>> > common code that is tested, but whatever), removed the old unused files, set<br>> > the svnprops and summoned the holy gods of the SVN branches to look over me.<br>> ><br>> > Attached is a SVN diff that should turn an up-to-date checkout of the trunk<br>> > into using the new GUI, hopefully without breaking anything. Apparently<br>> > Subversion is set to suck today, so the patch doesn't really remove the old<br>> > unused files (which are manually removed with svn rm on my local<br>> > repository).<br>> ><br>> > Whatever, after applying that patch those files are just ignored on compile<br>> > time, so... There goes the 24h countdown; I'm hoping that Max (and hopefully<br>> > some more awesome peeps) can apply the patch before tomorrow and make sure<br>> > it doesn't trash their branch.<br>> ><br>> ><br>> ><br>> ><br>> ><br>> ><br>> ><br>> > On Oct 6, 2008, at 2:25 PM, Max Horn wrote:<br>> ><br>> >> Vicent,<br>> >><br>> >> I can reproduce your merge problems. Very weird indeed. No idea what<br>> >> is causing them. Nor how to fix it...<br>> >><br>> >> Instead of wasting time trying to figure this one out, I propose that<br>> >> we instead perform a manual merge. This should be fine given that your<br>> >> changes are restricted to a small number of files:<br>> >><br>> >> Most stuff in gui/, as well as these (according to svnmerge's failed<br>> >> merge attempt):<br>> >><br>> >> C dists/msvc9/scumm.vcproj<br>> >> C dists/msvc9/scummvm.vcproj<br>> >> C engines/scumm/dialogs.cpp<br>> >> C engines/drascula/talk.cpp<br>> >> C engines/dialogs.cpp<br>> >> C engines/parallaction/saveload.cpp<br>> >> M engines/parallaction/objects.h<br>> >> C common/unzip.cpp<br>> >> M common/module.mk<br>> >> C + common/xmlparser.cpp<br>> >> M common/rect.h<br>> >> C + common/xmlparser.h<br>> >> A + graphics/VectorRenderer.h<br>> >> A + graphics/VectorRendererSpec.h<br>> >> C graphics/module.mk<br>> >> A + graphics/VectorRenderer.cpp<br>> >> A + graphics/VectorRendererSpec.cpp<br>> >> M graphics/font.cpp<br>> >> M base/main.cpp<br>> >><br>> >><br>> >> Assuming that your repos is synced with latest trunk, a manual merge<br>> >> could be as simple as "copy all modified files over to trunk, svn add<br>> >> new files, svn rm obsolete files". Then of course some test runs,<br>> >> maybe eyeballing of the "svn diff", and then just commit it...<br>> >><br>> >><br>> >> Cheers,<br>> >> Max<br>> >><br>> >> -------------------------------------------------------------------------<br>> >> This SF.Net email is sponsored by the Moblin Your Move Developer's<br>> >> challenge<br>> >> Build the coolest Linux based applications with Moblin SDK & win great<br>> >> prizes<br>> >> Grand prize is a trip for two to an Open Source event anywhere in the<br>> >> world<br>> >><span class="Apple-converted-space"> </span><a href="http://moblin-contest.org/redirect.php?banner_id=100&url=/">http://moblin-contest.org/redirect.php?banner_id=100&url=/</a><br>> >> _______________________________________________<br>> >> Scummvm-devel mailing list<br>> >><span class="Apple-converted-space"> </span><a href="mailto:Scummvm-devel@lists.sourceforge.net">Scummvm-devel@lists.sourceforge.net</a><br>> >><span class="Apple-converted-space"> </span><a href="https://lists.sourceforge.net/lists/listinfo/scummvm-devel">https://lists.sourceforge.net/lists/listinfo/scummvm-devel</a><br>> ><br>> ><br>> ><br>><span class="Apple-converted-space"> </span><br>> -------------------------------------------------------------------------<br>> This SF.Net email is sponsored by the Moblin Your Move Developer's challenge<br>> Build the coolest Linux based applications with Moblin SDK & win great prizes<br>> Grand prize is a trip for two to an Open Source event anywhere in the world<br>><span class="Apple-converted-space"> </span><a href="http://moblin-contest.org/redirect.php?banner_id=100&url=/">http://moblin-contest.org/redirect.php?banner_id=100&url=/</a><br>> _______________________________________________<br>> Scummvm-devel mailing list<br>><span class="Apple-converted-space"> </span><a href="mailto:Scummvm-devel@lists.sourceforge.net">Scummvm-devel@lists.sourceforge.net</a><br>><span class="Apple-converted-space"> </span><a href="https://lists.sourceforge.net/lists/listinfo/scummvm-devel">https://lists.sourceforge.net/lists/listinfo/scummvm-devel</a><br><br><hr>See how Windows connects the people, information, and fun that are part of your life.<span class="Apple-converted-space"> </span><a href="http://clk.atdmt.com/MRT/go/msnnkwxp1020093175mrt/direct/01/" target="_new">See Now</a></div></span></blockquote></div><br></body></html>