* fhandler_termios.cc (fhandler_termios::line_edit): Use special case '%0c'

handling to print non-printable characters using hex notation.
* smallprint.cc (__small_vsprintf): Semi-reimplement printing of non-printable
characters in hex but only when padding is specified.
* dcrt0.cc (dll_crt0_0): Remove tty_list initialization.
* shared.cc (memory_init): Initialize tty_list here.
* path.cc (path_conv::check): Remove unneeded parentheses from if check.

diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog
index f862a52e164b5f8a5954563c2e5b9e77f0beeccb..b66434600ededa21d8ca7226507473bc5ed1c3db 100644 (file)
--- a/winsup/cygwin/ChangeLog
+++ b/winsup/cygwin/ChangeLog
@@ -1,3 +1,16 @@
+2012-07-29  Christopher Faylor  <me.cygwin2012@cgf.cx>
+
+       * fhandler_termios.cc (fhandler_termios::line_edit): Use special case
+       '%0c' handling to print non-printable characters using hex notation.
+       * smallprint.cc (__small_vsprintf): Semi-reimplement printing of
+       non-printable characters in hex but only when padding is specified.
+
+       * dcrt0.cc (dll_crt0_0): Remove tty_list initialization.
+       * shared.cc (memory_init): Initialize tty_list here.
+
+       * path.cc (path_conv::check): Remove unneeded parentheses from if
+       check.
+
 2012-07-28  Corinna Vinschen  <corinna@vinschen.de>
 
        * include/inttypes.h: Add x86_64 target considerations throughout.

diff --git a/winsup/cygwin/dcrt0.cc b/winsup/cygwin/dcrt0.cc
index c1df0a2f660937352180b47311dec63f21490ee9..7e32de0efad171ede5d275cf9f8c837a55df1bed 100644 (file)
--- a/winsup/cygwin/dcrt0.cc
+++ b/winsup/cygwin/dcrt0.cc
@@ -774,7 +774,6 @@ dll_crt0_0 ()
   user_data->threadinterface->Init ();
 
   _cygtls::init ();
-  tty_list::init_session ();
 
   _main_tls = &_my_tls;
 

diff --git a/winsup/cygwin/fhandler_termios.cc b/winsup/cygwin/fhandler_termios.cc
index 690f08fa2befc117eb00540b9e7fcca9842829a8..18afacf1d139e64fcaae2e2d051d77a90436ac7d 100644 (file)
--- a/winsup/cygwin/fhandler_termios.cc
+++ b/winsup/cygwin/fhandler_termios.cc
@@ -237,7 +237,7 @@ fhandler_termios::line_edit (const char *rptr, int nread, termios& ti)
     {
       c = *rptr++;
 
-      termios_printf ("char %c", c);
+      paranoid_printf ("char %0c", c);
 
       /* Check for special chars */
 

diff --git a/winsup/cygwin/path.cc b/winsup/cygwin/path.cc
index 0a6cad76c70360da859d506c3518770f3a82ad5f..e67c91f17f2fc9d26acf132ffc47abb672f5dd00 100644 (file)
--- a/winsup/cygwin/path.cc
+++ b/winsup/cygwin/path.cc
@@ -1153,7 +1153,7 @@ out:
   if (opt & PC_CTTY)
     path_flags |= PATH_CTTY;
 
-  if ((opt & PC_POSIX))
+  if (opt & PC_POSIX)
     {
       if (tail < path_end && tail > path_copy + 1)
        *tail = '/';

diff --git a/winsup/cygwin/shared.cc b/winsup/cygwin/shared.cc
index 7a0b2510b1a3d5967e0ca2e7c82cc687800c3987..83402a2c8b9ff32ec29cac8d0bbcd1c191845e1b 100644 (file)
--- a/winsup/cygwin/shared.cc
+++ b/winsup/cygwin/shared.cc
@@ -377,4 +377,8 @@ memory_init (bool init_cygheap)
 
   shared_info::create ();      /* Initialize global shared memory */
   user_info::create (false);   /* Initialize per-user shared memory */
+  /* Initialize tty list session stuff.  Doesn't really belong here but
+     this needs to be initialized before any tty or console manipulation
+     happens and it is a common location.  */
+  tty_list::init_session ();
 }