dynamic_images.h doesn't build on gcc 3

[GoogleCodeExporter]

In file included from
/Users/luser/build/mozilla/toolkit/airbag/airbag/src/client/mac/handler/dynamic_
images.cc:35:
/Users/luser/build/mozilla/toolkit/airbag/airbag/src/client/mac/handler/dynamic_
images.h:
In
   member function `void google_breakpad::DynamicImage::Print()':
/Users/luser/build/mozilla/toolkit/airbag/airbag/src/client/mac/handler/dynamic_
images.h:156:
error: parse
   error before `;' token
/Users/luser/build/mozilla/toolkit/airbag/airbag/src/client/mac/handler/dynamic_
images.h:
In
   member function `void google_breakpad::DynamicImages::TestPrint()':
/Users/luser/build/mozilla/toolkit/airbag/airbag/src/client/mac/handler/dynamic_
images.h:281:
warning: invalid
   conversion from `int' to `google_breakpad::MachHeader*'
/Users/luser/build/mozilla/toolkit/airbag/airbag/src/client/mac/handler/dynamic_
images.h:281:
error: parse
   error before `;' token
/Users/luser/build/mozilla/toolkit/airbag/airbag/src/client/mac/handler/dynamic_
images.h:281:
warning: unused
   variable `google_breakpad::MachHeader*_dyld_get_image_header'

It looks like gcc 3 fails on the constructs:
MachHeader(*GetMachHeader()).Print();
MachHeader(*_dyld_get_image_header(i)).Print();
It looks like it's the dereferencing of the function return value that
fails.  This short patch fixes it.

Original issue reported on code.google.com by ted.mielczarek on 4 May 2007 at 2:10

Attachments:

• dynamic_images_gcc3.patch

[GoogleCodeExporter]

Needs an extra const to compile on x86.

Original comment by ted.mielczarek on 4 May 2007 at 3:00

Attachments:

• dynamic_images_gcc3.2.patch

[GoogleCodeExporter]

Original comment by ted.mielczarek on 4 May 2007 at 7:37

• Changed state: Fixed