Hi,
I propose the following attached patches:
Patch 1 removes the wrappers. I don't see why we need them. Also removes CD_FLAG_FILE, we have CD_FLAG_BIN which is the same.
Patches 2 and 3 simplify save_dump_dir_from_problem_data() code and the method it uses to pass back the name of the created directory. Also, move its decl into problem_data.h - I feel it belongs here.
These are pretty mechanical changes, which open up the way for the next step:
We need to merge create_dump_dir_from_problem_data() and save_dump_dir_from_problem_data() into one function, and also reuse this new function instead of open-coded problem_data -> dump_dir code in abrt-server. (IOW: we have triple code duplication right now, need to consolidate).