Date: Sat, 19 Nov 2016 11:30:14 +0000 (UTC) From: "Tobias C. Berner" <tcberner@FreeBSD.org> To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r426356 - head/multimedia/kdenlive/files Message-ID: <201611191130.uAJBUEoV024151@repo.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: tcberner Date: Sat Nov 19 11:30:13 2016 New Revision: 426356 URL: https://svnweb.freebsd.org/changeset/ports/426356 Log: Fix build of multimedia/kdenlive -- ambiguity of abs() Approved by: antoine Added: head/multimedia/kdenlive/files/ head/multimedia/kdenlive/files/patch-src_scopes_audioscopes_spectrogram.cpp (contents, props changed) Added: head/multimedia/kdenlive/files/patch-src_scopes_audioscopes_spectrogram.cpp ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/multimedia/kdenlive/files/patch-src_scopes_audioscopes_spectrogram.cpp Sat Nov 19 11:30:13 2016 (r426356) @@ -0,0 +1,37 @@ +Fix ambiguity of abs(). + +[7%] Building CXX object src/CMakeFiles/kdenlive.dir/scopes/audioscopes/spectrogram.cpp.o +/tmp/usr/ports/multimedia/kdenlive/work/kdenlive-16.04.3/src/scopes/audioscopes/spectrogram.cpp:276:77: error: call to 'abs' is ambiguous + hideText = m_aTrackMouse->isChecked() && m_mouseWithinWidget && abs(x-(leftDist + mouseX + 30)) < (int) minDistX + ^~~ +/usr/include/stdlib.h:83:6: note: candidate function +int abs(int) __pure2; + ^ +/usr/include/c++/v1/stdlib.h:115:44: note: candidate function +inline _LIBCPP_INLINE_VISIBILITY long abs( long __x) _NOEXCEPT {return labs(__x);} + ^ +/usr/include/c++/v1/stdlib.h:117:44: note: candidate function +inline _LIBCPP_INLINE_VISIBILITY long long abs(long long __x) _NOEXCEPT {return llabs(__x);} + ^ +[...] + +--- src/scopes/audioscopes/spectrogram.cpp.orig 2016-07-12 21:44:03 UTC ++++ src/scopes/audioscopes/spectrogram.cpp +@@ -245,7 +245,7 @@ QImage Spectrogram::renderHUD(uint) + x = leftDist + (m_innerScopeRect.width()-1) * ((float)hz)/m_freqMax; + + // Hide text if it would overlap with the text drawn at the mouse position +- hideText = m_aTrackMouse->isChecked() && m_mouseWithinWidget && abs(x-(leftDist + mouseX + 20)) < (int) minDistX + 16 ++ hideText = m_aTrackMouse->isChecked() && m_mouseWithinWidget && abs((int)(x-(leftDist + mouseX + 20))) < (int) minDistX + 16 + && mouseX < m_innerScopeRect.width() && mouseX >= 0; + + if (x <= rightBorder) { +@@ -273,7 +273,7 @@ QImage Spectrogram::renderHUD(uint) + } + // Draw the line at the very right (maximum frequency) + x = leftDist + m_innerScopeRect.width()-1; +- hideText = m_aTrackMouse->isChecked() && m_mouseWithinWidget && abs(x-(leftDist + mouseX + 30)) < (int) minDistX ++ hideText = m_aTrackMouse->isChecked() && m_mouseWithinWidget && abs((int)(x-(leftDist + mouseX + 30))) < (int) minDistX + && mouseX < m_innerScopeRect.width() && mouseX >= 0; + davinci.drawLine(x, topDist, x, topDist + m_innerScopeRect.height()+6); + if (!hideText) {
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201611191130.uAJBUEoV024151>