Remove 'GLOBAL int flags' completely!

It no longer has any flags in it at all, so its day is done.

pscp.c

@@ -2235,7 +2235,6 @@ int psftp_main(int argc, char *argv[])
 
     default_protocol = PROT_TELNET;
 
-    flags = 0;
     cmdline_tooltype = TOOLTYPE_FILETRANSFER;
     sk_init();
 

psftp.c

@@ -2769,7 +2769,6 @@ int psftp_main(int argc, char *argv[])
     bool sanitise_stderr = true;
     char *batchfile = NULL;
 
-    flags = 0;
     cmdline_tooltype = TOOLTYPE_FILETRANSFER;
     sk_init();
 

putty.h

@@ -578,19 +578,6 @@ extern const int be_default_protocol;
  */
 extern const char *const appname;
 
-/*
- * Some global flags denoting the type of application.
- *
- * These flags describe the type of _application_ - they wouldn't
- * vary between individual sessions - and so it's OK to have this
- * variable be GLOBAL.
- *
- * Note that additional flags may be defined in platform-specific
- * headers. It's probably best if those ones start from 0x1000, to
- * avoid collision.
- */
-GLOBAL int flags;
-
 /*
  * Likewise, these two variables are set up when the application
  * initialises, and inform all default-settings accesses after

unix/uxplink.c

@@ -603,7 +603,6 @@ int main(int argc, char **argv)
     stderr_bs = BinarySink_UPCAST(&stderr_bcs);
     outgoingeof = EOF_NO;
 
-    flags = 0;
     cmdline_tooltype |=
         (TOOLTYPE_HOST_ARG |
          TOOLTYPE_HOST_ARG_CAN_BE_SESSION |

windows/winpgnt.c

@@ -1183,8 +1183,6 @@ void cleanup_exit(int code)
     exit(code);
 }
 
-int flags = 0;
-
 struct winpgnt_client {
     PageantListenerClient plc;
 };

windows/winplink.c

@@ -269,7 +269,6 @@ int main(int argc, char **argv)
     default_protocol = PROT_SSH;
     default_port = 22;
 
-    flags = 0;
     cmdline_tooltype |=
         (TOOLTYPE_HOST_ARG |
          TOOLTYPE_HOST_ARG_CAN_BE_SESSION |