Project

General

Profile

Feature #822

Warnings while compiling with g++4.5

Added by Sebastien Gilles over 9 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Robin Mills
Category:
build
Target version:
0.24
Start date:
27 Apr 2012
Due date:
% Done:

0%

Estimated time:

Description

Hi,

I've just compiled exiv2 because I'm using it for one of my projects I want to run on multiple platforms (development was under a Debian and embedded exiv2 was perfectly fine, but there is no such possibility with Windows)

I compiled it with g++ 4.5, and there were plenty of warnings (with proper warning flags such as -Wall)

Most of them were about the deprecation of std::auto_ptr, but others were about entirely others topics (for instance dubious conversions from float to unsigned int)

Do you intend to correct these warnings? I didn't look into the code, but from my experience g++ warnings are very often relevant.

Thanks anyway for your work on your very useful library.

Best regards,

Sébastien Gilles

History

#1

Updated by Robin Mills over 9 years ago

  • Assignee set to Robin Mills
#2

Updated by Robin Mills over 9 years ago

  • Status changed from New to Assigned
#3

Updated by Robin Mills about 9 years ago

  • Status changed from Assigned to Resolved

The instructions for building on cygwin are: http://clanmills.com/exiv2/cygwin.shtml

I've recommended:
./configure --disable-visibility

as this will eliminate the visibility warnings. (Some Unix/GCC platforms require visibility)

I've built and run the 0.23 download from: http://www.exiv2.org/download.html with the following gcc:

512 rmills@rmills-laptop:/c/gnu.cygwin/exiv2 $ gcc --version
gcc (GCC) 4.5.3
513 rmills@rmills-laptop:~ $ uname -a
CYGWIN_NT-6.1-WOW64 rmills-laptop 1.7.15(0.260/5/3) 2012-05-09 10:25 i686 Cygwin
514 rmills@rmills-laptop:~ $

This compiles, links and executes our test suite with no errors or warnings.

I've submitted a change (which will be in 0.24) to ensure --disable-visibility is the default for cygwin.

#4

Updated by Robin Mills over 8 years ago

  • Status changed from Resolved to Closed
  • Target version set to 0.24

Fixed in 0.24.

Also available in: Atom PDF