Adds inline specifier to itertools callable objects in headers

Ryan Haining · Ryan Haining · commit ab4999407f73 · 2024-06-21T11:54:29.000-07:00
Related to issue #104
diff --git a/cppitertools/accumulate.hpp b/cppitertools/accumulate.hpp
@@ -17,7 +17,7 @@ namespace iter {
 
     using AccumulateFn = IterToolFnOptionalBindSecond<Accumulator, std::plus<>>;
   }
-  constexpr impl::AccumulateFn accumulate{};
+  inline constexpr impl::AccumulateFn accumulate{};
 }
 
 template <typename Container, typename AccumulateFunc>
diff --git a/cppitertools/batched.hpp b/cppitertools/batched.hpp
@@ -20,7 +20,7 @@ namespace iter {
 
     using BatchedFn = IterToolFnBindSizeTSecond<Batcher>;
   }
-  constexpr impl::BatchedFn batched{};
+  inline constexpr impl::BatchedFn batched{};
 }
 
 template <typename Container>
diff --git a/cppitertools/chunked.hpp b/cppitertools/chunked.hpp
@@ -20,7 +20,7 @@ namespace iter {
 
     using ChunkedFn = IterToolFnBindSizeTSecond<Chunker>;
   }
-  constexpr impl::ChunkedFn chunked{};
+  inline constexpr impl::ChunkedFn chunked{};
 }
 
 template <typename Container>
diff --git a/cppitertools/combinations.hpp b/cppitertools/combinations.hpp
@@ -15,7 +15,7 @@ namespace iter {
 
     using CombinationsFn = IterToolFnBindSizeTSecond<Combinator>;
   }
-  constexpr impl::CombinationsFn combinations{};
+  inline constexpr impl::CombinationsFn combinations{};
 }
 
 template <typename Container>
diff --git a/cppitertools/combinations_with_replacement.hpp b/cppitertools/combinations_with_replacement.hpp
@@ -15,7 +15,7 @@ namespace iter {
     using CombinationsWithReplacementFn =
         IterToolFnBindSizeTSecond<CombinatorWithReplacement>;
   }
-  constexpr impl::CombinationsWithReplacementFn combinations_with_replacement{};
+  inline constexpr impl::CombinationsWithReplacementFn combinations_with_replacement{};
 }
 
 template <typename Container>
diff --git a/cppitertools/cycle.hpp b/cppitertools/cycle.hpp
@@ -15,7 +15,7 @@ namespace iter {
 
     using CycleFn = IterToolFn<Cycler>;
   }
-  constexpr impl::CycleFn cycle{};
+  inline constexpr impl::CycleFn cycle{};
 }
 
 template <typename Container>
diff --git a/cppitertools/dropwhile.hpp b/cppitertools/dropwhile.hpp
@@ -16,7 +16,7 @@ namespace iter {
 
     using DropWhileFn = IterToolFnOptionalBindFirst<Dropper, BoolTester>;
   }
-  constexpr impl::DropWhileFn dropwhile{};
+  inline constexpr impl::DropWhileFn dropwhile{};
 }
 
 template <typename FilterFunc, typename Container>
diff --git a/cppitertools/enumerate.hpp b/cppitertools/enumerate.hpp
@@ -33,7 +33,7 @@ namespace iter {
 
     using EnumerateFn = IterToolFnOptionalBindSecond<Enumerable, std::size_t>;
   }
-  constexpr impl::EnumerateFn enumerate{};
+  inline constexpr impl::EnumerateFn enumerate{};
 }
 
 namespace std {
diff --git a/cppitertools/filter.hpp b/cppitertools/filter.hpp
@@ -24,7 +24,7 @@ namespace iter {
     using FilterFn = IterToolFnOptionalBindFirst<Filtered, BoolTester>;
   }
 
-  constexpr impl::FilterFn filter{};
+  inline constexpr impl::FilterFn filter{};
 }
 
 template <typename FilterFunc, typename Container>
diff --git a/cppitertools/filterfalse.hpp b/cppitertools/filterfalse.hpp
@@ -38,7 +38,7 @@ namespace iter {
 
     using FilterFalseFn = IterToolFnOptionalBindFirst<FilterFalsed, BoolTester>;
   }
-  constexpr impl::FilterFalseFn filterfalse{};
+  inline constexpr impl::FilterFalseFn filterfalse{};
 }
 
 // Delegates to Filtered with PredicateFlipper<FilterFunc>
diff --git a/cppitertools/groupby.hpp b/cppitertools/groupby.hpp
@@ -20,7 +20,7 @@ namespace iter {
 
     using GroupByFn = IterToolFnOptionalBindSecond<GroupProducer, Identity>;
   }
-  constexpr impl::GroupByFn groupby{};
+  inline constexpr impl::GroupByFn groupby{};
 }
 
 template <typename Container, typename KeyFunc>
diff --git a/cppitertools/imap.hpp b/cppitertools/imap.hpp
@@ -21,7 +21,7 @@ namespace iter {
       using PipeableAndBindFirst<IMapFn>::operator();
     };
   }
-  constexpr impl::IMapFn imap{};
+  inline constexpr impl::IMapFn imap{};
 }
 
 #endif
diff --git a/cppitertools/permutations.hpp b/cppitertools/permutations.hpp
@@ -17,7 +17,7 @@ namespace iter {
     class Permuter;
     using PermutationsFn = IterToolFn<Permuter>;
   }
-  constexpr impl::PermutationsFn permutations{};
+  inline constexpr impl::PermutationsFn permutations{};
 }
 
 template <typename Container>
diff --git a/cppitertools/powerset.hpp b/cppitertools/powerset.hpp
@@ -18,7 +18,7 @@ namespace iter {
 
     using PowersetFn = IterToolFn<Powersetter>;
   }
-  constexpr impl::PowersetFn powerset{};
+  inline constexpr impl::PowersetFn powerset{};
 }
 
 template <typename Container>
diff --git a/cppitertools/reversed.hpp b/cppitertools/reversed.hpp
@@ -45,7 +45,7 @@ namespace iter {
 
     using ReversedFn = IterToolFn<Reverser>;
   }
-  constexpr impl::ReversedFn reversed{};
+  inline constexpr impl::ReversedFn reversed{};
 }
 
 template <typename Container>
diff --git a/cppitertools/slice.hpp b/cppitertools/slice.hpp
@@ -172,7 +172,7 @@ struct iter::impl::SliceFn {
 };
 
 namespace iter {
-  constexpr impl::SliceFn slice{};
+  inline constexpr impl::SliceFn slice{};
 }
 
 #endif
diff --git a/cppitertools/sliding_window.hpp b/cppitertools/sliding_window.hpp
@@ -16,7 +16,7 @@ namespace iter {
     class WindowSlider;
     using SlidingWindowFn = IterToolFnBindSizeTSecond<WindowSlider>;
   }
-  constexpr impl::SlidingWindowFn sliding_window{};
+  inline constexpr impl::SlidingWindowFn sliding_window{};
 }
 
 template <typename Container>
diff --git a/cppitertools/sorted.hpp b/cppitertools/sorted.hpp
@@ -15,7 +15,7 @@ namespace iter {
     class SortedView;
     using SortedFn = IterToolFnOptionalBindSecond<SortedView, std::less<>>;
   }
-  constexpr impl::SortedFn sorted{};
+  inline constexpr impl::SortedFn sorted{};
 }
 
 template <typename Container, typename CompareFunc>
diff --git a/cppitertools/starmap.hpp b/cppitertools/starmap.hpp
@@ -252,7 +252,7 @@ struct iter::impl::StarMapFn : PipeableAndBindFirst<StarMapFn> {
 };
 
 namespace iter {
-  constexpr impl::StarMapFn starmap{};
+  inline constexpr impl::StarMapFn starmap{};
 }
 
 #endif
diff --git a/cppitertools/takewhile.hpp b/cppitertools/takewhile.hpp
@@ -16,7 +16,7 @@ namespace iter {
 
     using TakeWhileFn = IterToolFnOptionalBindFirst<Taker, BoolTester>;
   }
-  constexpr impl::TakeWhileFn takewhile{};
+  inline constexpr impl::TakeWhileFn takewhile{};
 }
 
 template <typename FilterFunc, typename Container>
diff --git a/cppitertools/unique_everseen.hpp b/cppitertools/unique_everseen.hpp
@@ -45,7 +45,7 @@ namespace iter {
     };
   }
 
-  constexpr impl::UniqueEverseenFn unique_everseen{};
+  inline constexpr impl::UniqueEverseenFn unique_everseen{};
 }
 
 #endif
diff --git a/cppitertools/unique_justseen.hpp b/cppitertools/unique_justseen.hpp
@@ -23,7 +23,7 @@ namespace iter {
       }
     };
   }
-  constexpr impl::UniqueJustseenFn unique_justseen{};
+  inline constexpr impl::UniqueJustseenFn unique_justseen{};
 }
 
 #endif

Original file line number	Diff line number	Diff line change
`@@ -17,7 +17,7 @@ namespace iter {`
`17`	`17`
`18`	`18`	`using AccumulateFn = IterToolFnOptionalBindSecond<Accumulator, std::plus<>>;`
`19`	`19`	`}`
`20`		`- constexpr impl::AccumulateFn accumulate{};`
	`20`	`+ inline constexpr impl::AccumulateFn accumulate{};`
`21`	`21`	`}`
`22`	`22`
`23`	`23`	`template <typename Container, typename AccumulateFunc>`
Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,7 @@ namespace iter {`
`20`	`20`
`21`	`21`	`using BatchedFn = IterToolFnBindSizeTSecond<Batcher>;`
`22`	`22`	`}`
`23`		`- constexpr impl::BatchedFn batched{};`
	`23`	`+ inline constexpr impl::BatchedFn batched{};`
`24`	`24`	`}`
`25`	`25`
`26`	`26`	`template <typename Container>`
Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ namespace iter {`
`15`	`15`
`16`	`16`	`using CombinationsFn = IterToolFnBindSizeTSecond<Combinator>;`
`17`	`17`	`}`
`18`		`- constexpr impl::CombinationsFn combinations{};`
	`18`	`+ inline constexpr impl::CombinationsFn combinations{};`
`19`	`19`	`}`
`20`	`20`
`21`	`21`	`template <typename Container>`
Original file line number	Diff line number	Diff line change
`@@ -16,7 +16,7 @@ namespace iter {`
`16`	`16`
`17`	`17`	`using DropWhileFn = IterToolFnOptionalBindFirst<Dropper, BoolTester>;`
`18`	`18`	`}`
`19`		`- constexpr impl::DropWhileFn dropwhile{};`
	`19`	`+ inline constexpr impl::DropWhileFn dropwhile{};`
`20`	`20`	`}`
`21`	`21`
`22`	`22`	`template <typename FilterFunc, typename Container>`
Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ namespace iter {`
`33`	`33`
`34`	`34`	`using EnumerateFn = IterToolFnOptionalBindSecond<Enumerable, std::size_t>;`
`35`	`35`	`}`
`36`		`- constexpr impl::EnumerateFn enumerate{};`
	`36`	`+ inline constexpr impl::EnumerateFn enumerate{};`
`37`	`37`	`}`
`38`	`38`
`39`	`39`	`namespace std {`
Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,7 @@ namespace iter {`
`24`	`24`	`using FilterFn = IterToolFnOptionalBindFirst<Filtered, BoolTester>;`
`25`	`25`	`}`
`26`	`26`
`27`		`- constexpr impl::FilterFn filter{};`
	`27`	`+ inline constexpr impl::FilterFn filter{};`
`28`	`28`	`}`
`29`	`29`
`30`	`30`	`template <typename FilterFunc, typename Container>`
Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ namespace iter {`
`38`	`38`
`39`	`39`	`using FilterFalseFn = IterToolFnOptionalBindFirst<FilterFalsed, BoolTester>;`
`40`	`40`	`}`
`41`		`- constexpr impl::FilterFalseFn filterfalse{};`
	`41`	`+ inline constexpr impl::FilterFalseFn filterfalse{};`
`42`	`42`	`}`
`43`	`43`
`44`	`44`	`// Delegates to Filtered with PredicateFlipper<FilterFunc>`