Skip to content

Commit

Permalink
textproc/clucene: apply gentoo patch for C++17 conformance
Browse files Browse the repository at this point in the history
Some parts of CLucene use std::binary_function, which was deprecated in
C++11, and has been completely removed as of C++17. This shows up while
building libreoffice with clang >= 16 or gcc >= 11, because C++17 is the
default standard now.

Apply a diff from https://bugs.gentoo.org/869170 that fixes this, by
removing usage of std::binary_function, and replacing it with typedefs.

PR:		271846
Approved by:	fluffy (maintainer)
MFH:		2023Q2
  • Loading branch information
DimitryAndric committed Jun 5, 2023
1 parent 01600e6 commit 36efbbe
Show file tree
Hide file tree
Showing 2 changed files with 148 additions and 1 deletion.
2 changes: 1 addition & 1 deletion textproc/clucene/Makefile
@@ -1,6 +1,6 @@
PORTNAME= clucene
PORTVERSION= 2.3.3.4
PORTREVISION= 20
PORTREVISION= 21
CATEGORIES= textproc
MASTER_SITES= SF/${PORTNAME}/${PORTNAME}-core-unstable/2.3
DISTNAME= ${PORTNAME}-core-${PORTVERSION}
Expand Down
147 changes: 147 additions & 0 deletions textproc/clucene/files/patch-fix-binary-function
@@ -0,0 +1,147 @@
Replace std::binary_function with typedefs (deprecated in c++11 and removed in c++17).
Bug: https://bugs.gentoo.org/869170
--- src/core/CLucene/index/_Term.h
+++ src/core/CLucene/index/_Term.h
@@ -13,9 +13,12 @@
CL_NS_DEF(index)


-class Term_Equals:public CL_NS_STD(binary_function)<const Term*,const Term*,bool>
+class Term_Equals
{
public:
+ using first_argument_type = const Term*;
+ using second_argument_type = const Term*;
+ using result_type = bool;
bool operator()( const Term* val1, const Term* val2 ) const{
return val1->equals(val2);
}
--- src/core/CLucene/search/BooleanQuery.cpp
+++ src/core/CLucene/search/BooleanQuery.cpp
@@ -25,9 +25,12 @@ CL_NS_USE(index)
CL_NS_USE(util)
CL_NS_DEF(search)

- class BooleanClause_Compare:public CL_NS_STD(binary_function)<const BooleanClause*,const BooleanClause*,bool>
+ class BooleanClause_Compare
{
public:
+ using first_argument_type = const BooleanClause*;
+ using second_argument_type = const BooleanClause*;
+ using result_type = bool;
bool operator()( const BooleanClause* val1, const BooleanClause* val2 ) const {
return val1->equals(val2);
}
--- src/core/CLucene/search/MultiPhraseQuery.cpp
+++ src/core/CLucene/search/MultiPhraseQuery.cpp
@@ -377,9 +377,12 @@ TCHAR* MultiPhraseQuery::toString(const TCHAR* f) const {
return buffer.giveBuffer();
}

-class TermArray_Equals:public CL_NS_STD(binary_function)<const Term**,const Term**,bool>
+class TermArray_Equals
{
public:
+ using first_argument_type = const Term**;
+ using second_argument_type = const Term**;
+ using result_type = bool;
bool operator()( CL_NS(util)::ArrayBase<CL_NS(index)::Term*>* val1, CL_NS(util)::ArrayBase<CL_NS(index)::Term*>* val2 ) const{
if ( val1->length != val2->length )
return false;
--- src/core/CLucene/util/Equators.h
+++ src/core/CLucene/util/Equators.h
@@ -22,21 +22,30 @@ CL_NS_DEF(util)
/** @internal */
class CLUCENE_INLINE_EXPORT Equals{
public:
- class CLUCENE_INLINE_EXPORT Int32:public CL_NS_STD(binary_function)<const int32_t*,const int32_t*,bool>
+ class CLUCENE_INLINE_EXPORT Int32
{
public:
+ using first_argument_type = const int32_t*;
+ using second_argument_type = const int32_t*;
+ using result_type = bool;
bool operator()( const int32_t val1, const int32_t val2 ) const;
};

- class CLUCENE_INLINE_EXPORT Char:public CL_NS_STD(binary_function)<const char*,const char*,bool>
+ class CLUCENE_INLINE_EXPORT Char
{
public:
+ using first_argument_type = const char*;
+ using second_argument_type = const char*;
+ using result_type = bool;
bool operator()( const char* val1, const char* val2 ) const;
};
#ifdef _UCS2
- class CLUCENE_INLINE_EXPORT WChar: public CL_NS_STD(binary_function)<const wchar_t*,const wchar_t*,bool>
+ class CLUCENE_INLINE_EXPORT WChar
{
public:
+ using first_argument_type = const wchar_t*;
+ using second_argument_type = const wchar_t*;
+ using result_type = bool;
bool operator()( const wchar_t* val1, const wchar_t* val2 ) const;
};
class CLUCENE_INLINE_EXPORT TChar: public WChar{
@@ -48,9 +57,12 @@ public:


template<typename _cl>
- class CLUCENE_INLINE_EXPORT Void:public CL_NS_STD(binary_function)<const void*,const void*,bool>
+ class CLUCENE_INLINE_EXPORT Void
{
public:
+ using first_argument_type = const void*;
+ using second_argument_type = const void*;
+ using result_type = bool;
bool operator()( _cl* val1, _cl* val2 ) const{
return val1==val2;
}
--- src/core/CLucene/util/_Arrays.h
+++ src/core/CLucene/util/_Arrays.h
@@ -124,12 +124,14 @@ CL_NS_DEF(util)

template <typename _kt, typename _comparator,
typename class1, typename class2>
- class CLListEquals:
- public CL_NS_STD(binary_function)<class1*,class2*,bool>
+ class CLListEquals
{
typedef typename class1::const_iterator _itr1;
typedef typename class2::const_iterator _itr2;
public:
+ using first_argument_type = class1*;
+ using second_argument_type = class2*;
+ using result_type = bool;
CLListEquals(){
}
bool equals( class1* val1, class2* val2 ) const{
--- src/test/index/TestTermVectorsReader.cpp
+++ src/test/index/TestTermVectorsReader.cpp
@@ -93,17 +93,21 @@ CL_NS_USE(util);
}
};

- struct MyTCharCompare :
- public std::binary_function<const TCHAR*, const TCHAR*, bool>
+ struct MyTCharCompare
{
+ using first_argument_type = const TCHAR*;
+ using second_argument_type = const TCHAR*;
+ using result_type = bool;
bool operator () (const TCHAR* v1, const TCHAR* v2) const {
return _tcscmp(v1, v2) < 0;
}
};

- struct TestTokenCompare :
- public std::binary_function<const TestToken*, const TestToken*, bool>
+ struct TestTokenCompare
{
+ using first_argument_type = const TestToken*;
+ using second_argument_type = const TestToken*;
+ using result_type = bool;
bool operator () (const TestToken* t1, const TestToken* t2) const {
return t1->pos < t2->pos;
}

0 comments on commit 36efbbe

Please sign in to comment.