do not use reinterpret_cast 

do not use reinterpret_cast 

do not declare C-style arrays, use std::array<> instead 

do not declare C-style arrays, use std::array<> instead 

do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead 

do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead 

assigning newly created 'gsl::owner<>' to non-owner 'FILE *' (aka '_IO_FILE *') 

the value returned by this function should be used 

calling legacy resource function without passing a 'gsl::owner<>' 

do not use reinterpret_cast 

do not use reinterpret_cast 

function 'create_snapshot' has cognitive complexity of 41 (threshold 25) 

'data' should be used for accessing the data pointer instead of taking the address of the 0-th element 

'data' should be used for accessing the data pointer instead of taking the address of the 0-th element 

initializing non-owner 'FILE *' (aka '_IO_FILE *') with a newly created 'gsl::owner<>' 

the value returned by this function should be used 

calling legacy resource function without passing a 'gsl::owner<>' 

deleting a pointer through a type that is not marked 'gsl::owner<>'; consider using a smart pointer instead 

initializing non-owner 'FILE *' (aka '_IO_FILE *') with a newly created 'gsl::owner<>' 

calling legacy resource function without passing a 'gsl::owner<>' 

function 'update_min_epoch_id' has cognitive complexity of 32 (threshold 25) 

initializing non-owner 'FILE *' (aka '_IO_FILE *') with a newly created 'gsl::owner<>' 

calling legacy resource function without passing a 'gsl::owner<>' 

function 'begin_backup' has cognitive complexity of 28 (threshold 25)