merge lcov separately, to be able to pass separate binary-path to grcov for unit tests and python binding

Author: arvidn

.github/workflows/coverage.yml

@@ -52,6 +52,7 @@ jobs:
         python -m pip install colorama
         python -m pip install maturin
         python -m pip install pytest
+        sudo apt install lcov
 
         rustup target add x86_64-unknown-linux-musl
         cargo install grcov
@@ -66,8 +67,14 @@ jobs:
       run: |
         . ./activate
         LLVM_PROFILE_FILE="pytest.profraw" pytest clvm/tests
+        grcov . -s . --binary-path ./venv/lib/python3.9/site-packages/ -t lcov --branch --ignore-not-existing -o pytest.lcov.info --keep-only src
+        rm pytest.profraw
+
         LLVM_PROFILE_FILE="unittest.profraw" RUSTFLAGS="-Zinstrument-coverage" cargo test --no-default-features
-        grcov . -s . --binary-path ./venv/lib/python3.9/site-packages/ -t lcov --branch --ignore-not-existing -o lcov.info --keep-only src
+        grcov . -s . --binary-path ./target/debug/build -t lcov --branch --ignore-not-existing -o unittest.lcov.info --keep-only src
+
+        # merge the lcov files
+        lcov unittest.lcov.info pytest.lcov.info -o lcov.info
 
     - name: Coveralls
       uses: coverallsapp/github-action@master

src/int_allocator.rs

@@ -29,12 +29,6 @@ pub struct IntAllocator {
     atom_vec: Vec<IntAtomBuf>,
 }
 
-impl Default for IntAllocator {
-    fn default() -> Self {
-        Self::new()
-    }
-}
-
 impl IntAllocator {
     pub fn new() -> Self {
         let mut r = IntAllocator {