bbitmap: report size_used in buffered dump path

Return size_used for both complete and incomplete buffered dumps.

Reported-by: CodexGPT-5.3 <noreply@openai.com>

Author: msune

CHANGELOG.md

@@ -17,11 +17,13 @@ Legend:
 - [B] str: fix `cdada_str_replace*()` to reject empty match and self-replacement issues
 - [B] str: return `NULL` in `cdada_str_create()` when input C string is `NULL`
 - [B] str: align `cdada_str()` invalid-handle behavior with API docs (return `NULL`, not empty string)
+- [B] bbitmap: make `cdada_bbitmap_dump()` set `size_used` in buffered paths too
 - [O] tests: add coverage for `cdada_strerr()` valid, boundary and high invalid values
 - [O] tests: add `oom_create_test` coverage for create paths (`list`, `map`, `queue`, `set`, `stack`, `str`, `bbitmap`)
 - [O] tests: add coverage for `cdada_str_replace_all()` empty-match invalid input and self-overlap replacement
 - [O] tests: add `cdada_str_create(NULL)` regression coverage
 - [O] tests: align invalid `cdada_str()` assertion with documented `NULL` return
+- [O] tests: validate `cdada_bbitmap_dump()` updates `size_used` for both incomplete and complete buffered dumps
 
 ### v0.6.3 (28th January 2026)
 

src/bbitmap.cc

@@ -112,15 +112,16 @@ int cdada_bbitmap_dump(cdada_bbitmap_t* b, uint32_t size, char* buffer,
 		std::stringstream ss;
 		for(i=0;i<b->n_words; ++i)
 			__cdada_bbitmap_dump_word(i, &b->ptr[i], ss);
+		*size_used = ss.str().length()+1;
 
-		if(!buffer){
-			*size_used = ss.str().length()+1;
+		if(!buffer)
 			return CDADA_SUCCESS;
-		}
 
 		snprintf(buffer, size, "%s", ss.str().c_str());
-		if(ss.str().length()+1 > size)
+		if(*size_used > size){
+			*size_used = size;
 			return CDADA_E_INCOMPLETE;
+		}
 	}catch(bad_alloc& e){
 		return CDADA_E_MEM;
 	}catch(...){

test/bbitmap_test.c

@@ -95,11 +95,13 @@ int test_basics(){
 
 	rv = cdada_bbitmap_dump(b, 128, buffer, &used);
 	TEST_ASSERT(rv == CDADA_E_INCOMPLETE);
+	TEST_ASSERT(used == 128);
 	fprintf(stderr, "%s\n", buffer);
 
 	char buffer2[2048*16];
 	rv = cdada_bbitmap_dump(b, 2048*16, buffer2, &used);
 	TEST_ASSERT(rv == CDADA_SUCCESS);
+	TEST_ASSERT(used > 128);
 	fprintf(stderr, "%s\n", buffer2);
 	rv = cdada_bbitmap_print(b, stdout);
 	TEST_ASSERT(rv == CDADA_SUCCESS);