On 3/12/2013 1:40 PM, Yang Tse wrote:
On Tue, Mar 12, 2013, Tom Grace <[email protected]> wrote:

Also I see in TOOL_MAIN.C   that variable vms_show  is a module static yet ,
the thread update comments suggest it should be extern, via edit submitted
in TOOL_MAIN.H   ,  what do you want to do here?

(there is compile warning for this)

Tom, copy & paste full warning including line reference.

Not sure what happened, either I missed something in the patch submission or something else went wrong. It definitely needs to be extern or the VMS exit code does not work.

I will try to find time to check all the files in my local directory against what was checked in.

This is what tool_main.c needs to be for VMS.

--- /src_Root/curl/src/tool_main.c      Wed Feb  6 03:47:19 2013
+++ /vms_root/curl/src/tool_main.c      Sat Jan 19 21:31:06 2013
@@ -27,6 +27,10 @@
 #include <signal.h>
 #endif

+#ifdef HAVE_UNISTD_H
+#  include <unistd.h>
+#endif
+
 #define ENABLE_CURLX_PRINTF
 /* use our own printf() functions */
 #include "curlx.h"
@@ -44,16 +48,6 @@
  * anyway for convenience.
  */
 #include "memdebug.h" /* keep this as LAST include */
-
-#ifdef __VMS
-/*
- * vms_show is a global variable, used in main() as parameter for
- * function vms_special_exit() to allow proper curl tool exiting.
- * Its value may be set in other tool_*.c source files thanks to
- * forward declaration present in tool_vms.h
- */
-static int vms_show = 0;
-#endif

 /*
  * Ensure that file descriptors 0, 1 and 2 (stdin, stdout, stderr) are





-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html

Reply via email to