On Tuesday 26 of June 2012 12:34:45 Jiri Moskovcak wrote:
> ---
> src/include/dump_dir.h | 1 +
> src/lib/dump_dir.c | 10 ++++++++++
> 2 files changed, 11 insertions(+)
>
> diff --git a/src/include/dump_dir.h b/src/include/dump_dir.h
> index f8d2931..1073949 100644
> --- a/src/include/dump_dir.h
> +++ b/src/include/dump_dir.h
> @@ -70,6 +70,7 @@ void dd_save_text(struct dump_dir *dd, const char *name,
> const char *data); void dd_save_binary(struct dump_dir *dd, const char
> *name, const char *data, unsigned size); /* Returns 0 if directory is
> deleted or not found */
> int dd_delete(struct dump_dir *dd);
> +int dd_rename(struct dump_dir *dd, const char *new_path);
>
>
> /* reported_to handling */
> diff --git a/src/lib/dump_dir.c b/src/lib/dump_dir.c
> index 908a2df..a61fccf 100644
> --- a/src/lib/dump_dir.c
> +++ b/src/lib/dump_dir.c
> @@ -913,6 +913,16 @@ report_result_t *find_in_reported_to(struct dump_dir
> *dd, const char *prefix) return result;
> }
>
> +int dd_rename(struct dump_dir *dd, const char *new_path)
> +{
> + int res = rename(dd->dd_dirname, new_path);
> + if (res == 0)
> + {
> + free(dd->dd_dirname);
> + dd->dd_dirname = rm_trailing_slashes(new_path);
> + }
> + return res;
> +}
>
> /* Utility function */
Looks good to me.
Note : I miss comments for almost all dd_* functions :), I know that functions
are self explanatory, but ...
Regards,
Jakub
