Feature #822

Warnings while compiling with g++4.5

Added by Sebastien Gilles over 5 years ago. Updated about 4 years ago.

Status:ClosedStart date:27 Apr 2012
Priority:NormalDue date:
Assignee:Robin Mills% Done:

0%

Category:build
Target version:0.24

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 5 years ago

  • Assignee set to Robin Mills

#2 Updated by Robin Mills over 5 years ago

  • Status changed from New to Assigned

#3 Updated by Robin Mills about 5 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 about 4 years ago

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

Fixed in 0.24.

Also available in: Atom PDF

Redmine Appliance - Powered by TurnKey Linux