
<html>

<head>

<style>

.hmmessage P

{

margin:0px;

padding:0px

}

body.hmmessage

{

font-size: 10pt;

font-family:Verdana

}

</style>

</head>

<body class='hmmessage'>

> Date: Thu, 7 May 2009 22:17:48 +0200<br>> From: lordhoto@gmail.com<br>> To: max@quendi.de<br>> CC: philipk79@hotmail.com; scummvm-devel@lists.sourceforge.net<br>> Subject: Re: [Scummvm-devel] [Scummvm-cvs-logs] SF.net SVN: scummvm:[40351] scummvm/trunk/sound/softsynth/opl/opl.h<br>> <br>> Max Horn wrote:<br>> ><br>> > Am 07.05.2009 um 21:38 schrieb Johannes Schickel:<br>> ><br>> >> Filippos Karapetis wrote:<br>> >>> In that case, we can just remove the files from the VS project files<br>> >>> so that they won't be compiled straight away. Thus, the .cpp file does<br>> >>> not need to be renamed...<br>> >><br>> >> I thought about that too, but if someone might want to edit the code via<br>> >> VS that would be much more inconvenient than just renaming the files<br>> >> IMHO. Also it protects other builds envs from doing the same mistake<br>> >> (XCode projects maybe).<br>> ><br>> > With any decent IDE I used in the past (XCode, Eclipse, Codewarrior), <br>> > it was possible to add .cpp files and mark them as "do not compile", <br>> > just to facilitate uses exactly like this. Surely VS also can do that?<br>> ><br>> <br>> I don't know to be honest, I thought so too. I guess the main problem <br>> was that nowhere it is mentioned that the file is only a 'template' for <br>> code creation. I don't like this template thing too much though, my plan <br>> is to (eventually) get rid of it in the long run. Of course doing <br>> branches instead of the #ifdefs might have impacts on execution speed. <br>> For now I would be fine with renaming them back to .h/.cpp and adding a <br>> slight notice about this fact in the .cpp file, of course I hope people <br>> updating project files will read such notes :-).<br>> <br>> // Johannes<br><br><br>Indeed, there is an option to flag a file so it won't be compiled in

MSVC. I renamed the two files back to opl.h and opl.cpp, marking

opl.cpp as "do not compile", and it compiles fine under MSVC. Should I

committhis?<br><br>Filippos<br><br><br /><hr />Hotmail® goes with you.  <a href='http://windowslive.com/Tutorial/Hotmail/Mobile?ocid=TXT_TAGLM_WL_HM_Tutorial_Mobile1_052009' target='_new'>Get it on your BlackBerry or iPhone.</a></body>

</html>