Make the implementation of find_next() consistent with that of find_next_off()

gennaroprota · gennaroprota · commit caee95bddd8b · 2025-10-15T11:41:44.000+02:00
diff --git a/include/boost/dynamic_bitset/impl/dynamic_bitset.ipp b/include/boost/dynamic_bitset/impl/dynamic_bitset.ipp
@@ -1442,10 +1442,9 @@ template< typename Block, typename AllocatorOrContainer >
 BOOST_DYNAMIC_BITSET_CONSTEXPR20 typename dynamic_bitset< Block, AllocatorOrContainer >::size_type
 dynamic_bitset< Block, AllocatorOrContainer >::find_next( size_type pos ) const
 {
-    if ( pos == npos ) {
-        return npos;
-    }
-    return find_first( pos + 1 );
+    return pos == npos
+             ? npos
+             : find_first( pos + 1 );
 }
 
 template< typename Block, typename AllocatorOrContainer >

Original file line number	Diff line number	Diff line change
`@@ -1442,10 +1442,9 @@ template< typename Block, typename AllocatorOrContainer >`
`1442`	`1442`	`BOOST_DYNAMIC_BITSET_CONSTEXPR20 typename dynamic_bitset< Block, AllocatorOrContainer >::size_type`
`1443`	`1443`	`dynamic_bitset< Block, AllocatorOrContainer >::find_next( size_type pos ) const`
`1444`	`1444`	`{`
`1445`		`- if ( pos == npos ) {`
`1446`		`- return npos;`
`1447`		`- }`
`1448`		`- return find_first( pos + 1 );`
	`1445`	`+ return pos == npos`
	`1446`	`+ ? npos`
	`1447`	`+ : find_first( pos + 1 );`
`1449`	`1448`	`}`
`1450`	`1449`
`1451`	`1450`	`template< typename Block, typename AllocatorOrContainer >`