Address several issues. c_is_fnchar must be namespace protected [for

  linkage, not for external use], and this patch expands its use to
  include IS_SHCHAR() tests for apr_thread_proc.c in Win32.

  Also introduce apr_proc_t ->invoked member --- for logging or audit
  purposes alone, for platforms that create a single 'command' string.
  This doesn't adversly impact portability [have APR_HAS_PROC_INVOKED
  feature macro to assure other platforms don't struggle with it] but
  helps us measure portability when attempting to emulate execxx()
  behavior on Win32 [and perhaps OS2 at some point.)


git-svn-id: http://svn.apache.org/repos/asf/apr/apr/trunk@63178 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 9 insertions, 3 deletions

diff --git a/include/apr_thread_proc.h b/include/apr_thread_proc.h
index 5bbc0ede3..a722756a4 100644
--- a/include/apr_thread_proc.h
+++ b/include/apr_thread_proc.h
@@ -160,9 +160,15 @@ struct apr_proc_t {
     apr_file_t *out;
     /** Parent's side of pipe to child's stdouterr */
     apr_file_t *err;
-#ifdef WIN32
-    /** Must retain the handle as any clone may not have the
-     *  the same permissions 
+#if APR_HAS_PROC_INVOKED || defined(DOXYGEN)
+    /** Diagnositics/debugging string of the command invoked for 
+     *  this process [only present if APR_HAS_PROC_INVOKED is true]
+     */
+    char *invoked;
+#endif
+#if defined(WIN32) || defined(DOXYGEN)
+    /** Win32 specific: Must retain the creator's handle granting 
+     *  access, as a new copy may not grant the same permissions 
      */
     HANDLE hproc;
 #endif