I noticed this cast in the committed code:
> + num_total_db = get_dbname_oid_list_from_mfile((char *) inputFileSpec,
> &dbname_oid_list);
The cast drops the const qualifier from inputFileSpec.
get_dbname_oid_list_from_mfile() writes into the space pointed to by its
argument, so it's really not "const". (And inputFileSpec points into
argv, so this ends up writing directly into argv.)
Please see if you can clean this up. It might be best if
get_dbname_oid_list_from_mfile() made a copy of its argument that it can
write into, and then the argument can be "const".
