Hi, I'll send a patch in a separate message.
One more thing: I tried to compile the sources with clang, basically just setting it up with CC=clang ./configure Works fine, gives a few more compiler warnings on possible problems than gcc does. The encode/decode demos work as expected. Regards, Bertrik Sikken On 29-1-2014 12:16, David Rowe wrote: > Hello Bertrik, > > Thanks I have fixed the SVN cp paths. ampext.c was some experimental > code that's not used in Codec 2, so it's probably not worth maintaining. > > Happy to look at any other issues code tools throw up. > > Cheers, > > David > > On Tue, 2014-01-28 at 23:25 +0100, Bertrik Sikken wrote: >> Hi, >> >> I just noticed that http://codec2.org/ points to an empty web page, is >> that OK? >> >> Also, the rowetel page about codec2 points to source archive >> https://freetel.svn.sourceforge.net/svnroot/freetel/codec2-dev >> but the correct one seems to be >> https://svn.code.sf.net/p/freetel/code/codec2-dev >> >> Finally, I ran the tool 'cppcheck' on the source code. Cppcheck is a >> static code analysis tool for the C and C++ programming languages. >> Cppcheck found a few programming errors, for example a buffer overflow in >> src/ampext.c line 723 >> I won't mind helping a bit with clearing those up. >> >> With kind regards, >> Bertrik Sikken ------------------------------------------------------------------------------ Managing the Performance of Cloud-Based Applications Take advantage of what the Cloud has to offer - Avoid Common Pitfalls. Read the Whitepaper. http://pubads.g.doubleclick.net/gampad/clk?id=121051231&iu=/4140/ostg.clktrk _______________________________________________ Freetel-codec2 mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/freetel-codec2
