Skip to content

Commit

Permalink
Fix issues found by Boosts inspect tool
Browse files Browse the repository at this point in the history
  • Loading branch information
apolukhin committed Jul 7, 2020
1 parent 3a36467 commit 077ea54
Show file tree
Hide file tree
Showing 7 changed files with 29 additions and 3 deletions.
3 changes: 2 additions & 1 deletion include/boost/pfr/detail/detectors.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,8 @@

namespace boost { namespace pfr { namespace detail {
///////////////////// `value` is true if Detector<Tleft, Tright> does not compile (SFINAE)
struct success{};
struct success{};

template <template <class, class> class Detector, class Tleft, class Tright>
struct not_appliable {
static constexpr bool value = std::is_same<
Expand Down
4 changes: 2 additions & 2 deletions test/common/test_tuple_sizes_on.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -58,8 +58,8 @@ void test_counts_on_multiple_chars_impl() {
static_assert(flat_tuple_size_v<std::conditional_t<std::is_fundamental<T1>::value, T1*, void*> > == 1, "");
#if !defined(__GNUC__) || __GNUC__ != 8
// GCC-8 has big problems with this test:
// error: constexpr ubiq_constructor::operator Type&() const [with Type = test_counts_on_multiple_chars()::t2*],
// declared using local type test_counts_on_multiple_chars()::t2, is used but never defined [-fpermissive]
// error: 'constexpr ubiq_constructor::operator Type&() const [with Type = test_counts_on_multiple_chars()::t2*]',
// declared using local type 'test_counts_on_multiple_chars()::t2', is used but never defined [-fpermissive]
//
// Fixed in GCC-9.
static_assert(tuple_size_v<T1*> == 1, "");
Expand Down
5 changes: 5 additions & 0 deletions test/flat/flat_motivating_example.cpp
Original file line number Diff line number Diff line change
@@ -1,3 +1,8 @@
// Copyright (c) 2016-2020 Antony Polukhin
//
// Distributed under the Boost Software License, Version 1.0. (See accompanying
// file LICENSE_1_0.txt or copy at http://www.boost.org/LICENSE_1_0.txt)

// requires: C++14
#include <iostream>
#include "boost/pfr.hpp"
Expand Down
5 changes: 5 additions & 0 deletions test/flat/flat_motivating_example2.cpp
Original file line number Diff line number Diff line change
@@ -1,3 +1,8 @@
// Copyright (c) 2016-2020 Antony Polukhin
//
// Distributed under the Boost Software License, Version 1.0. (See accompanying
// file LICENSE_1_0.txt or copy at http://www.boost.org/LICENSE_1_0.txt)

#include <iostream>
#include "boost/pfr.hpp"

Expand Down
5 changes: 5 additions & 0 deletions test/precise/motivating_example.cpp
Original file line number Diff line number Diff line change
@@ -1,3 +1,8 @@
// Copyright (c) 2016-2020 Antony Polukhin
//
// Distributed under the Boost Software License, Version 1.0. (See accompanying
// file LICENSE_1_0.txt or copy at http://www.boost.org/LICENSE_1_0.txt)

// requires: C++14
#include <iostream>
#include "boost/pfr/precise.hpp"
Expand Down
5 changes: 5 additions & 0 deletions test/precise/motivating_example0.cpp
Original file line number Diff line number Diff line change
@@ -1,3 +1,8 @@
// Copyright (c) 2016-2020 Antony Polukhin
//
// Distributed under the Boost Software License, Version 1.0. (See accompanying
// file LICENSE_1_0.txt or copy at http://www.boost.org/LICENSE_1_0.txt)

#include <iostream>
#include <string>
#include "boost/pfr/precise.hpp"
Expand Down
5 changes: 5 additions & 0 deletions test/precise/motivating_example2.cpp
Original file line number Diff line number Diff line change
@@ -1,3 +1,8 @@
// Copyright (c) 2016-2020 Antony Polukhin
//
// Distributed under the Boost Software License, Version 1.0. (See accompanying
// file LICENSE_1_0.txt or copy at http://www.boost.org/LICENSE_1_0.txt)

#include <iostream>
#include "boost/pfr/precise.hpp"

Expand Down

0 comments on commit 077ea54

Please sign in to comment.