patch for gcc4

Help
2005-10-16
2013-04-30
  • I was having problems getting cdrdao to compile with gcc4, getting an error in the trackdb directory for FormatConverter.h. I found the below patch here:

    http://www.mail-archive.com/pld-cvs-commit@lists.pld-linux.org/msg13498.html

    ...the important part is the two 'class Toc;' declarations in CueParser.cc and FormatConverter.h. It fixed the issue.

    It would be great if someone would integrate this into the code!

    Peter

    patch below:

    Author: areq                         Date: Sat Sep 17 13:49:49 2005 GMT
    Module: SOURCES                       Tag: HEAD
    ---- Log message:
    - gcc4 fix

    ---- Files affected:
    SOURCES:
       cdrdao-gcc4.patch (NONE -> 1.1)  (NEW)

    ---- Diffs:

    ================================================================
    Index: SOURCES/cdrdao-gcc4.patch
    diff -u /dev/null SOURCES/cdrdao-gcc4.patch:1.1
    --- /dev/null   Sat Sep 17 15:49:49 2005
    +++ SOURCES/cdrdao-gcc4.patch   Sat Sep 17 15:49:43 2005
    @@ -0,0 +1,24 @@
    +diff -Nur cdrdao-1.2.0.org/trackdb/CueParser.cc
    cdrdao-1.2.0/trackdb/CueParser.cc
    +--- cdrdao-1.2.0.org/trackdb/CueParser.cc      2005-05-09 21:21:28.000000000
    +0200
    ++++ cdrdao-1.2.0/trackdb/CueParser.cc  2005-09-17 15:46:15.638476312 +0200
    +@@ -23,6 +23,8 @@
    +
    + #include "Cue2Toc.h"
    +
    ++class Toc;
    ++
    + extern Toc *parseToc(const char* tocBuffer, const char *filename);
    +
    + Toc *parseCue(FILE *fp, const char *filename)
    +diff -Nur cdrdao-1.2.0.org/trackdb/FormatConverter.h
    cdrdao-1.2.0/trackdb/FormatConverter.h
    +--- cdrdao-1.2.0.org/trackdb/FormatConverter.h 2005-04-22 04:01:46.000000000
    +0200
    ++++ cdrdao-1.2.0/trackdb/FormatConverter.h     2005-09-17 15:45:27.509792984
    +0200
    +@@ -27,6 +27,8 @@
    + #include "TrackData.h"
    + #include "Toc.h"
    +
    ++class Toc;
    ++
    + // Quick abstract class declarations. Format converters should derive
    + // their own FormatSupport and FormatSupportManager.
    +
    ================================================================

     
    • Denis Leroy
      Denis Leroy
      2005-10-18

      Thanks. This was already fixed in CVS.