#6 gcc4.4 fix

closed
Kelly Fitz
5
2010-02-15
2010-02-03
Holger
No

Hi,

I had to patch today's svn of loris because of a typical gcc4.4-error: cstdio was not included in FourierTransform.C

Patch attached. It also fixes a little typo

Cheers

Holger

Discussion

  • Holger
    Holger
    2010-02-03

    gcc44-and-typo-fix

     
    Attachments
  • Kelly Fitz
    Kelly Fitz
    2010-02-04

    The typo I see, but I am not seeing anything in FourierTransform that indicates cstudio is appropriate. What is the compiler error you are seeing in the absence of cstudio?

    Thanks for your help,
    Kelly

     
  • Kelly Fitz
    Kelly Fitz
    2010-02-04

    • status: open --> pending
    • assigned_to: nobody --> kfitz
     
  • Holger
    Holger
    2010-02-04

    gcc-4.4 error

     
    Attachments
  • Holger
    Holger
    2010-02-04

    • status: pending --> open
     
  • Holger
    Holger
    2010-02-04

    error.log attached.
    I found a similar error here ( http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=505013 ) and jumped to conclusions, but perhaps it's something completely different :-)

     
  • Kelly Fitz
    Kelly Fitz
    2010-02-04

    Thanks for the error log. Could you please try commenting out the line #137

    #include <fftw3.h>

    in FourierTransform.C

    and inserting

    #if defined(HAVE_FFTW3_H) && HAVE_FFTW3_H
    #include <fftw3.h>
    #endif

    at line 54, and let me know whether that also solves the problem? If so, then I understand the problem and will release a fix.

    Thanks again,
    Kelly

     
  • Holger
    Holger
    2010-02-04

    ok, this fixes it, too.... thanks for your patience! :-)

     
  • Kelly Fitz
    Kelly Fitz
    2010-02-15

    Resolved.

     
  • Kelly Fitz
    Kelly Fitz
    2010-02-15

    • status: open --> closed