Skip to content

Commit 863da9f

Browse files
authored
style(clang-tidy): Clean include headers of src/core/json (#1646)
Reported by clang-tidy check `misc-include-cleaner` Refs: sourcemeta/blaze#429 Signed-off-by: Balakrishna Avulapati <[email protected]>
1 parent 821218d commit 863da9f

File tree

8 files changed

+32
-13
lines changed

8 files changed

+32
-13
lines changed

src/core/json/include/sourcemeta/core/json.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55
#include <sourcemeta/core/json_export.h>
66
#endif
77

8-
#include <sourcemeta/core/json_hash.h>
98
#include <sourcemeta/core/json_value.h>
109

1110
#include <cstdint> // std::uint64_t

src/core/json/include/sourcemeta/core/json_error.h

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,8 @@
88
#include <cstdint> // std::uint64_t
99
#include <exception> // std::exception
1010
#include <filesystem> // std::filesystem::path
11+
#include <string> // std::string
12+
#include <utility> // std::move
1113

1214
namespace sourcemeta::core {
1315

src/core/json/include/sourcemeta/core/json_object.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33

44
#include <algorithm> // std::swap
55
#include <cassert> // assert
6-
#include <functional> // std::equal_to, std::less
6+
#include <cstddef> // std::size_t
77
#include <initializer_list> // std::initializer_list
88
#include <iterator> // std::advance
99
#include <utility> // std::pair, std::move

src/core/json/include/sourcemeta/core/json_value.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,11 +11,11 @@
1111

1212
#include <algorithm> // std::any_of
1313
#include <cassert> // assert
14+
#include <cstddef> // std::size_t
1415
#include <cstdint> // std::int64_t, std::uint8_t
1516
#include <functional> // std::less, std::reference_wrapper, std::function
1617
#include <initializer_list> // std::initializer_list
1718
#include <memory> // std::allocator
18-
#include <optional> // std::optional
1919
#include <set> // std::set
2020
#include <sstream> // std::basic_istringstream
2121
#include <string> // std::basic_string, std::char_traits

src/core/json/json.cc

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,15 @@
22
#include "stringify.h"
33

44
#include <sourcemeta/core/json.h>
5+
#include <sourcemeta/core/json_error.h>
6+
#include <sourcemeta/core/json_value.h>
57

68
#include <cassert> // assert
9+
#include <cstdint> // std::uint64_t
10+
#include <filesystem> // std::filesystem
711
#include <fstream> // std::ifstream
12+
#include <istream> // std::basic_istream
13+
#include <ostream> // std::basic_ostream
814
#include <system_error> // std::make_error_code, std::errc
915

1016
namespace sourcemeta::core {

src/core/json/json_value.cc

Lines changed: 15 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,20 @@
1+
#include <sourcemeta/core/json_array.h>
12
#include <sourcemeta/core/json_value.h>
23

3-
#include <algorithm> // std::find
4-
#include <cassert> // assert
5-
#include <cmath> // std::isinf, std::isnan, std::modf, std::trunc
6-
#include <numeric> // std::transform
7-
#include <stdexcept> // std::invalid_argument
8-
#include <string> // std::to_string
9-
#include <utility> // std::move
10-
#include <vector> // std::vector
4+
#include <algorithm> // std::find
5+
#include <cassert> // assert
6+
#include <cmath> // std::isinf, std::isnan, std::modf, std::trunc
7+
#include <cstddef> // std::size_t
8+
#include <cstdint> // std::int64_t
9+
#include <functional> // std::reference_wrapper
10+
#include <initializer_list> // std::initializer_list
11+
#include <numeric> // std::transform
12+
#include <sstream> // std::basic_istringstream
13+
#include <stdexcept> // std::invalid_argument
14+
#include <string> // std::to_string
15+
#include <string_view> // std::basic_string_view
16+
#include <utility> // std::move
17+
#include <vector> // std::vector
1118

1219
namespace sourcemeta::core {
1320

src/core/json/parser.h

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,11 +3,12 @@
33

44
#include "grammar.h"
55

6-
#include <sourcemeta/core/json.h>
76
#include <sourcemeta/core/json_error.h>
7+
#include <sourcemeta/core/json_value.h>
88

99
#include <cassert> // assert
1010
#include <cctype> // std::isxdigit
11+
#include <cstddef> // std::size_t
1112
#include <cstdint> // std::uint64_t
1213
#include <functional> // std::reference_wrapper
1314
#include <istream> // std::basic_istream

src/core/json/stringify.h

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,14 +3,18 @@
33

44
#include "grammar.h"
55

6-
#include <sourcemeta/core/json.h>
6+
#include <sourcemeta/core/json_value.h>
77

88
#include <algorithm> // std::transform, std::sort
9+
#include <cstddef> // std::size_t
10+
#include <cstdint> // std::int64_t
911
#include <iomanip> // std::setprecision
1012
#include <ios> // std::noshowpoint, std::fixed
1113
#include <iterator> // std::next, std::cbegin, std::cend, std::back_inserter
1214
#include <ostream> // std::basic_ostream
15+
#include <sstream> // std::ostringstream
1316
#include <string> // std::to_string
17+
#include <vector> // std::vector
1418

1519
namespace sourcemeta::core::internal {
1620
constexpr auto LINE_WIDTH{80};

0 commit comments

Comments
 (0)