Skip to content

Commit

Permalink
Aint need no C++17, C++14 would suffice
Browse files Browse the repository at this point in the history
  • Loading branch information
lisitsyn committed Feb 1, 2019
1 parent 1dc80c3 commit 2ddb189
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
project (Tapkee LANGUAGES CXX)

cmake_minimum_required (VERSION 3.8)
cmake_minimum_required (VERSION 3.5.1)

# set paths
set (CMAKE_CXX_STANDARD 17)
set (CMAKE_CXX_STANDARD 14)
set (TAPKEE_INCLUDE_DIR "${CMAKE_CURRENT_SOURCE_DIR}/include")
set (TAPKEE_SRC_DIR "${CMAKE_CURRENT_SOURCE_DIR}/src")
set (TAPKEE_TESTS_DIR "${CMAKE_CURRENT_SOURCE_DIR}/test/unit")
Expand Down
2 changes: 1 addition & 1 deletion include/tapkee/methods.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -161,7 +161,7 @@ class ImplementationBase
case X: \
{ \
timed_context tctx__("[+] embedding with " # X); \
if constexpr ( \
if ( \
((!MethodTraits<X>::needs_kernel) || (!is_dummy<KernelCallback>::value)) && \
((!MethodTraits<X>::needs_distance) || (!is_dummy<DistanceCallback>::value)) && \
((!MethodTraits<X>::needs_features) || (!is_dummy<FeaturesCallback>::value)) \
Expand Down

0 comments on commit 2ddb189

Please sign in to comment.