diff options
| author |  Thiago Macieira <thiago@macieira.org> | 2014-08-17 10:42:47 -0700 |
|---|---|---|
| committer |  Dirk Hohndel <dirk@hohndel.org> | 2014-08-17 12:01:56 -0600 |
| commit | f3d44db512f5e877065cbb0afdfd25ce206976dd (patch) | |
| tree | 62c39fcdd5ce14ec7c8e659c56f4bd2606645ac7 /save-html.h | |
| parent | 29f50c6aeeb31dfad7510d240e27905a9f62670d (diff) | |
| download | subsurface-f3d44db512f5e877065cbb0afdfd25ce206976dd.tar.gz | |
Fix and simplify get_file_name
This function created a QFile that it didn't need and it returned a
dangling pointer (the std::string object was destroyed at the end of the
function).
The function now returns a pointer that the caller must free. For that
reason, it's easier in copy_image_and_overwrite to just go for the
QFileInfo call.
Signed-off-by: Thiago Macieira <thiago@macieira.org>
Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'save-html.h')
| -rw-r--r-- | save-html.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/save-html.h b/save-html.h index 6d31e4fc3..f47ae3ba6 100644 --- a/save-html.h +++ b/save-html.h @@ -19,7 +19,7 @@ void export_HTML(const char *file_name, const char *photos_dir, const bool selec void export_translation(const char *file_name); extern void copy_image_and_overwrite(const char *cfileName, const char *cnewName); -extern const char *get_file_name(const char *fileName); +extern char *get_file_name(const char *fileName); #ifdef __cplusplus } #endif |