Project

General

Profile

Bug #809

abs ambiguity in nikonmn.cpp

Added by Pavel Heimlich almost 10 years ago. Updated over 9 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Andreas Huggel
Category:
miscellaneous
Target version:
0.23
Start date:
25 Jan 2012
Due date:
% Done:

100%

Estimated time:

Description

exiv2 0.22 fails to build with the below message:

gmake1: Entering directory `/var/tmp/exiv2/exiv2-0.22/src'
libtool: compile: /opt/test2/SolarisStudio12.3-solaris-x86-bin/solarisstudio12.3/bin/CC -library=stdcxx4 -lCrun -xldscope=hidden -I. "-DEXV_LOCALEDIR=\"/usr/local/share/locale\"" -I../xmpsdk/include -c -DEXV_BUILDING_LIB=1 nikonmn.cpp -KPIC -DPIC -o .libs/nikonmn.o
"nikonmn.cpp", line 2457: Error: Overloading ambiguity between "std::abs(double)" and "std::abs(float)".
"nikonmn.cpp", line 2458: Error: Overloading ambiguity between "std::abs(double)" and "std::abs(float)".

Solaris 11, Solaris Studio 12.3, x86

Files

exiv2-nikonmn.diff (863 Bytes) exiv2-nikonmn.diff patch Pavel Heimlich, 25 Jan 2012 14:40
T809.patch (588 Bytes) T809.patch Visual Studio patch for r2665 Thomas Beutlich, 27 Feb 2012 12:24

Associated revisions

Revision 2665 (diff)
Added by Andreas Huggel over 9 years ago

#809: Applied patch by Pavel Heimlich.

Revision 2667 (diff)
Added by Andreas Huggel over 9 years ago

#809: Applied patch by Thomas Beutlich to fix Windows ccompilation error.

History

#1

Updated by Andreas Huggel over 9 years ago

  • Status changed from New to Resolved
  • Assignee set to Andreas Huggel
  • Target version set to 0.23
  • % Done changed from 0 to 100

Applied the patch without any changes, thanks!

#2

Updated by Thomas Beutlich over 9 years ago

Patch does not compile on Visual Studio.

>  nikonmn.cpp
>..\..\src\nikonmn.cpp(2481): error C2220: warning treated as error - no 'object' file generated
>..\..\src\nikonmn.cpp(2481): warning C4244: 'initializing' : conversion from 'float' to 'long', possible loss of data
New patch is attached.

#3

Updated by Andreas Huggel over 9 years ago

Thanks Thomas, I've applied the patch.

#4

Updated by Andreas Huggel over 9 years ago

  • Status changed from Resolved to Closed

Also available in: Atom PDF