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 | |
parent | c1a32d7c6e87f3d81fa095e7e7fe11fa8d88fe5f (diff) | |
download | netsurf-975a1d3c5db0dc11a4f5ada68d5d424850858161.tar.gz netsurf-975a1d3c5db0dc11a4f5ada68d5d424850858161.tar.bz2 |
Fix cppcheck variableScope warnings
-rw-r--r-- | amiga/arexx.c | 6 | ||||
-rw-r--r-- | amiga/download.c | 8 |
2 files changed, 9 insertions, 5 deletions
diff --git a/amiga/arexx.c b/amiga/arexx.c index 7fc58ef97..3bac0c249 100644 --- a/amiga/arexx.c +++ b/amiga/arexx.c @@ -196,11 +196,12 @@ int ami_find_tab_bw(struct gui_window_2 *gwin, struct browser_window *bw) struct browser_window *ami_find_tab(int window, int tab) { - int windows = 0; struct nsObject *node, *nnode; if(!IsMinListEmpty(window_list)) { + int windows = 0; + node = (struct nsObject *)GetHead((struct List *)window_list); do @@ -568,7 +569,6 @@ STATIC VOID rx_windows(struct ARexxCmd *cmd, struct RexxMsg *rxm __attribute__(( STATIC VOID rx_active(struct ARexxCmd *cmd, struct RexxMsg *rxm __attribute__((unused))) { - int windows = 0; int window = 0, tab = 0; struct browser_window *bw = curbw; struct nsObject *node, *nnode; @@ -578,6 +578,8 @@ STATIC VOID rx_active(struct ARexxCmd *cmd, struct RexxMsg *rxm __attribute__((u if(!IsMinListEmpty(window_list)) { + int windows = 0; + node = (struct nsObject *)GetHead((struct List *)window_list); do 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); |