Thanks Tzafrir, I have checked in patches 2 and 4, which are the areas of the code I am responsible for.
Cheers, David On Mon, 2014-03-24 at 18:26 +0200, Tzafrir Cohen wrote: > I recently looked again at codec2. The number of build warnings annoyed > me. > > Patch1: > Why is in-place build disabled? Is there any other reason besides the > cookbook files? Patch1 removes this (I did not include the removal > of those files). > > Patch2: > Patch2 removes some of the build warnings. I don't call for its > application, but merely for someone to look at the warnings. > > There are also many warnings about unused functions. Some of them > seem to be dead code IMHO. We have version control for the older > history. > > Patch3: > At first I thought that the build failed, because at the very last line > of the build I saw "errors" (and one line above that: "errors" in red. > Patch 3 works around that, at the price of an unneeded dependency. Any > better ideas? > > Pathc4: > Patch4 is a case where the compiler warning about an unintilized > variable actually does indicate an uninitialized variable. > > > ------------------------------------------------------------------------------ > Learn Graph Databases - Download FREE O'Reilly Book > "Graph Databases" is the definitive new guide to graph databases and their > applications. Written by three acclaimed leaders in the field, > this first edition is now available. Download your free book today! > http://p.sf.net/sfu/13534_NeoTech > _______________________________________________ > Freetel-codec2 mailing list > [email protected] > https://lists.sourceforge.net/lists/listinfo/freetel-codec2 ------------------------------------------------------------------------------ Learn Graph Databases - Download FREE O'Reilly Book "Graph Databases" is the definitive new guide to graph databases and their applications. Written by three acclaimed leaders in the field, this first edition is now available. Download your free book today! http://p.sf.net/sfu/13534_NeoTech _______________________________________________ Freetel-codec2 mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/freetel-codec2
