From: Robert J. <rj...@sp...> - 2007-01-07 20:55:16
|
Hello again, On Sunday 07 January 2007 20:53, Wil...@gm... wrote: > Hallo Muse team, > > in my eyes it would be a good start getting in touch with the > muse source code would be an attempt to fix all compiler warnings. Now, that is a very good idea. I have often thought to go through them but never actually done it. Likely you will fix a bug or two in the process. Please do it! :) The question of doing it in muse-0.9 and HEAD is up to you. Since you are up and running in 0.9 that's probably easiest. In the long run I think we should strive to move to HEAD. Regards, Robert > > I snatched several warnings like these (while I compiled muse-0.9pre2): > > ----------------------------------------------------------------- > dummyaudio.cpp:245: warning: unused variable 'n' > audioprefetch.cpp:185: warning: comparison between signed and unsigned > integer expressions synth.h:75: warning: 'class SynthIF' has virtual > functions but non-virtual destructor sync.cpp:292: warning: converting to > 'int' from 'double' > tempo.cpp:309: warning: unused parameter 'sn' > midi.cpp:825: warning: array subscript has type 'char' > midi.cpp:812: warning: 'drumRecPitch' may be used uninitialized in this > function undo.cpp:202: warning: enumeration value 'ModifyClip' not handled > in switch ----------------------------------------------------------------- > > > I could go to the "0.9pre code" and modify these changes and send you a > "diff -ur" > > QUESTION: > Shall I do the modifications on the > muse-0.9pre2 source code branch > or on the > HEAD source code branch > > Yours, > Willy > > ------------------------------------------------------------------------- > Take Surveys. Earn Cash. Influence the Future of IT > Join SourceForge.net's Techsay panel and you'll get the chance to share > your opinions on IT & business topics through brief surveys - and earn cash > http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV > _______________________________________________ > Lmuse-developer mailing list > Lmu...@li... > https://lists.sourceforge.net/lists/listinfo/lmuse-developer -- http://spamatica.se/musicsite/ |