diff options
author | Chris Young <chris@unsatisfactorysoftware.co.uk> | 2014-04-05 19:56:28 +0100 |
---|---|---|
committer | Chris Young <chris@unsatisfactorysoftware.co.uk> | 2014-04-05 20:26:40 +0100 |
commit | 975a1d3c5db0dc11a4f5ada68d5d424850858161 (patch) | |
tree | 90c96abadaa01ce463edbbd7800a639b6ede62e0 /amiga/download.c | |
parent | c1a32d7c6e87f3d81fa095e7e7fe11fa8d88fe5f (diff) | |
download | netsurf-975a1d3c5db0dc11a4f5ada68d5d424850858161.tar.gz netsurf-975a1d3c5db0dc11a4f5ada68d5d424850858161.tar.bz2 |
Fix cppcheck variableScope warnings
Diffstat (limited to 'amiga/download.c')
-rw-r--r-- | amiga/download.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/amiga/download.c b/amiga/download.c index fb5c10957..74cb51f15 100644 --- a/amiga/download.c +++ b/amiga/download.c @@ -347,7 +347,6 @@ void ami_free_download_list(struct List *dllist) void gui_window_save_link(struct gui_window *g, const char *url, const char *title) { - BPTR fh = 0; char fname[1024]; STRPTR openurlstring,linkname; struct DiskObject *dobj = NULL; @@ -369,6 +368,8 @@ gui_window_save_link(struct gui_window *g, const char *url, const char *title) if(ami_download_check_overwrite(fname, g->shared->win, 0)) { + BPTR fh; + if(fh = FOpen(fname,MODE_NEWFILE,0)) { /* TODO: Should be URLOpen on OS4.1 */ @@ -401,8 +402,6 @@ BOOL ami_download_check_overwrite(const char *file, struct Window *win, ULONG si /* Return TRUE if file can be (over-)written */ int32 res = 0; BPTR lock = 0; - BPTR fh = 0; - int64 oldsize = 0; char *overwritetext; if(nsoption_bool(ask_overwrite) == false) return TRUE; @@ -412,6 +411,9 @@ BOOL ami_download_check_overwrite(const char *file, struct Window *win, ULONG si if(lock) { if(size) { + BPTR fh; + int64 oldsize = 0; + if(fh = OpenFromLock(lock)) { oldsize = GetFileSize(fh); Close(fh); |