Date: Fri, 18 May 2018 10:19:00 +0000 (UTC) From: Raphael Kubo da Costa <rakuco@FreeBSD.org> To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r470285 - head/print/lyx/files Message-ID: <201805181019.w4IAJ0Xl069236@repo.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: rakuco Date: Fri May 18 10:19:00 2018 New Revision: 470285 URL: https://svnweb.freebsd.org/changeset/ports/470285 Log: Add a patch to fix the build on FreeBSD 10.4. pkg-fallout reports the following error: TexRow.cpp:271:6: error: return type 'const TexRow::TextEntry' must match previous return type 'TexRow::TextEntry' when lambda expression has unspecified explicit return type return text_none; ^ TexRow.cpp:273:3: error: return type 'const TexRow::TextEntry' must match previous return type 'TexRow::TextEntry' when lambda expression has unspecified explicit return type return text_none; ^ TexRow.cpp:288:6: error: return type 'lyx::TexRow::TextEntry' must match previous return type 'const TexRow::TextEntry' when lambda expression has unspecified explicit return type return last_pos; ^ TexRow.cpp:290:6: error: return type 'struct TextEntry' must match previous return type 'const TexRow::TextEntry' when lambda expression has unspecified explicit return type return entry.text; ^ TexRow.cpp:293:3: error: return type 'lyx::TexRow::TextEntry' must match previous return type 'const TexRow::TextEntry' when lambda expression has unspecified explicit return type return last_pos; ^ Make the lambda return type explicit to work around what looks like a bug on an old clang version. Not bumping PORTREVISION, as this shouldn't change the code in FreeBSD versions with a more recent clang. Added: head/print/lyx/files/ head/print/lyx/files/patch-src_TexRow.cpp (contents, props changed) Added: head/print/lyx/files/patch-src_TexRow.cpp ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/print/lyx/files/patch-src_TexRow.cpp Fri May 18 10:19:00 2018 (r470285) @@ -0,0 +1,39 @@ +Fixes the build on 10.4 with clang 3.4.1. + + TexRow.cpp:271:6: error: return type 'const TexRow::TextEntry' must match previous return type 'TexRow::TextEntry' when lambda expression has unspecified explicit return type + return text_none; + ^ + TexRow.cpp:273:3: error: return type 'const TexRow::TextEntry' must match previous return type 'TexRow::TextEntry' when lambda expression has unspecified explicit return type + return text_none; + ^ + TexRow.cpp:288:6: error: return type 'lyx::TexRow::TextEntry' must match previous return type 'const TexRow::TextEntry' when lambda expression has unspecified explicit return type + return last_pos; + ^ + TexRow.cpp:290:6: error: return type 'struct TextEntry' must match previous return type 'const TexRow::TextEntry' when lambda expression has unspecified explicit return type + return entry.text; + ^ + TexRow.cpp:293:3: error: return type 'lyx::TexRow::TextEntry' must match previous return type 'const TexRow::TextEntry' when lambda expression has unspecified explicit return type + return last_pos; + ^ + +Not upstreamable, this seems to be a workaround for a compiler bug. +--- src/TexRow.cpp.orig 2018-02-25 00:11:18 UTC ++++ src/TexRow.cpp +@@ -260,7 +260,7 @@ TexRow::getEntriesFromRow(int const row) const + return {text_none, text_none}; + + // find the start entry +- TextEntry const start = [&]() { ++ TextEntry const start = [&]() -> TextEntry const { + for (size_t j = i; j > 0; --j) { + if (!isNone(rowlist_[j].getTextEntry())) + return rowlist_[j].getTextEntry(); +@@ -274,7 +274,7 @@ TexRow::getEntriesFromRow(int const row) const + } (); + + // find the end entry +- TextEntry end = [&]() { ++ TextEntry end = [&]() -> TextEntry const { + if (isNone(start)) + return text_none; + // select up to the last position of the starting paragraph as a
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201805181019.w4IAJ0Xl069236>