From 1f4bb413011768267242023fa66830f156951fa0 Mon Sep 17 00:00:00 2001 From: John Mark Bell Date: Fri, 18 Dec 2009 00:11:37 +0000 Subject: I've decided that the user probably doesn't care if setting the filetype of saved HTML documents to HTML fails or not. More importantly, this also squashes a memory leak, and stops returning false from a function which is defined as returning -1 on error. svn path=/trunk/netsurf/; revision=9731 --- riscos/save.c | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/riscos/save.c b/riscos/save.c index 44a2b1835..b4a897424 100644 --- a/riscos/save.c +++ b/riscos/save.c @@ -1042,14 +1042,16 @@ int save_complete_htmlSaveFileFormat(const char *path, const char *filename, return -1; } snprintf(fullpath, len, "%s.%s", path, filename); + ret = htmlSaveFileFormat(fullpath, cur, encoding, format); + error = xosfile_set_type(fullpath, 0xFAF); if (error) { LOG(("xosfile_save_stamped: 0x%x: %s", - error->errnum, error->errmess)); - warn_user("SaveError", error->errmess); - return false; + error->errnum, error->errmess)); + /* Don't warn the user here -- they probably don't care */ } + free(fullpath); return ret; } -- cgit v1.2.3