Add implicit signed/unsigned conversion casts

Author: flanglet

src/bitstream/DefaultInputBitStream.hpp

@@ -136,7 +136,7 @@ namespace kanzi {
        }
 
        // Regular processing, buffer length is multiple of 8
-       _current = BigEndian::readLong64(&_buffer[_position]);
+       _current = uint64(BigEndian::readLong64(&_buffer[_position]));
        _position += 8;
        return 64;
    }

src/bitstream/DefaultOutputBitStream.hpp

@@ -120,7 +120,7 @@ namespace kanzi
    // Push 64 bits of current value into buffer.
    inline void DefaultOutputBitStream::pushCurrent()
    {
-       BigEndian::writeLong64(&_buffer[_position], _current);
+       BigEndian::writeLong64(&_buffer[_position], int64(_current));
        _availBits = 64;
        _current = 0;
        _position += 8;
@@ -151,7 +151,7 @@ namespace kanzi
 
        // Flush buffer
        // Round down to byte alignment
-       const uint a = _availBits & -8;
+       const uint a = _availBits & ~7;
 
        for (uint i = 56; i >= a; i -= 8) {
           _buffer[_position++] = byte(_current >> i);

src/concurrent.hpp

@@ -78,7 +78,7 @@ class Task {
 #ifdef CONCURRENCY_ENABLED
    class ThreadPool FINAL {
    public:
-       ThreadPool(uint32_t threads = 8);
+       ThreadPool(int threads = 8);
        template<class F, class... Args>
 #if __cplusplus >= 201703L // result_of deprecated from C++17
        std::future<typename std::invoke_result_t<F, Args...>> schedule(F&& f, Args&&... args);
@@ -96,14 +96,14 @@ class Task {
    };
 
 
-   inline ThreadPool::ThreadPool(uint32_t threads)
+   inline ThreadPool::ThreadPool(int threads)
        :   _stop(false)
    {
        if ((threads == 0) || (threads > 1024))
            throw std::invalid_argument("The number of threads must be in [1..1024]");
 
        // Start and run threads
-       for (uint32_t i = 0; i < threads; i++)
+       for (int i = 0; i < threads; i++)
            _workers.emplace_back(
                [this]
                {

src/io/CompressedOutputStream.hpp

@@ -208,7 +208,7 @@ namespace kanzi {
                if (_bufferId + 1 < nbTasks) {
                    _bufferId++;
                    const int bSize = _blockSize + (_blockSize >> 6);
-                   const int bufSize = (bSize > 65536) ? bSize : 65536;
+                   const size_t bufSize = (bSize > 65536) ? bSize : 65536;
 
                    if (_buffers[_bufferId]->_length == 0) {
                        delete[] _buffers[_bufferId]->_array;

src/util/strings.hpp

@@ -115,7 +115,7 @@ inline int tokenizeCSV(std::string& s, std::vector<std::string>& tokens, char de
     if (tk.size() > 0)
         tokens.push_back(tk);
 
-    return tokens.size();
+    return int(tokens.size());
 }
 
 inline std::string formatSize(const std::string& input)