fix: functions visibility

Author: MathisGD

src/LogarithmicBuckets.sol

@@ -106,6 +106,8 @@ library LogarithmicBuckets {
         if (_buckets.buckets[_bucket].insert(_id, _head)) _buckets.bucketsMask |= _bucket;
     }
 
+    /// PURE HELPERS ///
+
     /// @notice Returns the bucket in which the given value would fall.
     function computeBucket(uint256 _value) internal pure returns (uint256) {
         uint256 lowerMask = setLowerBits(_value);

test/TestLogarithmicBuckets.t.sol

@@ -134,7 +134,7 @@ contract TestLogarithmicBuckets is LogarithmicBucketsMock, Test {
 
     function testProveNextBucket(uint256 _value) public {
         uint256 curr = LogarithmicBuckets.computeBucket(_value);
-        uint256 next = nextBucket(_value);
+        uint256 next = nextBucketValue(_value);
         uint256 bucketsMask = buckets.bucketsMask;
         // check that `next` is a strictly higer non-empty bucket, or zero
         assertTrue(next == 0 || isPowerOfTwo(next));
@@ -151,7 +151,7 @@ contract TestLogarithmicBuckets is LogarithmicBucketsMock, Test {
 
     function testProvePrevBucket(uint256 _value) public {
         uint256 curr = LogarithmicBuckets.computeBucket(_value);
-        uint256 prev = prevBucket(_value);
+        uint256 prev = prevBucketValue(_value);
         uint256 bucketsMask = buckets.bucketsMask;
         // check that `prev` is a non-empty bucket that is lower than or equal to `curr`; or zero
         assertTrue(prev == 0 || isPowerOfTwo(prev));

test/mocks/LogarithmicBucketsMock.sol

@@ -48,13 +48,13 @@ contract LogarithmicBucketsMock {
         return true;
     }
 
-    function nextBucket(uint256 _value) internal view returns (uint256) {
+    function nextBucketValue(uint256 _value) internal view returns (uint256) {
         uint256 bucketsMask = buckets.bucketsMask;
         uint256 lowerMask = LogarithmicBuckets.setLowerBits(_value);
         return LogarithmicBuckets.nextBucket(lowerMask, bucketsMask);
     }
 
-    function prevBucket(uint256 _value) internal view returns (uint256) {
+    function prevBucketValue(uint256 _value) internal view returns (uint256) {
         uint256 bucketsMask = buckets.bucketsMask;
         uint256 lowerMask = LogarithmicBuckets.setLowerBits(_value);
         return LogarithmicBuckets.prevBucket(lowerMask, bucketsMask);