• B
    Fix clang_analyzer failure (#8492) · 714ce504
    Baptiste Lemaire 提交于
    Summary:
    Previously, the following command:
    ```USE_CLANG=1 TEST_TMPDIR=/dev/shm/rocksdb OPT=-g make -j$(nproc) analyze```
    was raising an error/warning the new_mem could potentially be a `nullptr`. This error appeared due to code changes from https://github.com/facebook/rocksdb/issues/8454, including an if-statement containing "`... && new_mem != nullptr && ...`", which made the analyzer believe that past this `if`-statement, a `new_mem==nullptr` was a possible scenario.
    This code patch simply introduces `assert`s and removes this condition in the `if`-statement.
    
    Pull Request resolved: https://github.com/facebook/rocksdb/pull/8492
    
    Reviewed By: jay-zhuang
    
    Differential Revision: D29571275
    
    Pulled By: bjlemaire
    
    fbshipit-source-id: 75d72246b70ebbbae7dea11ccb5778686d8bcbea
    714ce504
db_impl_write.cc 85.2 KB