* exceptions.cc (try_to_debug): Improve comment.

* fhandler_tty.cc (fhandler_pty_master::fhandler_pty_master): Add a comment.
This commit is contained in:
Christopher Faylor 2010-10-24 15:26:05 +00:00
parent c4f296c2a9
commit 488ddaced3
3 changed files with 13 additions and 6 deletions

View File

@ -1,3 +1,9 @@
2010-10-24 Christopher Faylor <me+cygwin@cgf.cx>
* exceptions.cc (try_to_debug): Improve comment.
* fhandler_tty.cc (fhandler_pty_master::fhandler_pty_master): Add a
comment.
2010-10-23 Christopher Faylor <me+cygwin@cgf.cx>
* pinfo.cc (pinfo::exit): Grab a tty lock before setting pgid.

View File

@ -390,8 +390,9 @@ try_to_debug (bool waitloop)
suspend_all_threads_except (current_thread_id);
*/
/* if any of these mutexes is owned, we will fail to start any cygwin app
until trapped app exits */
/* If the tty mutex is owned, we will fail to start any cygwin app
until the trapped app exits. However, this will only release any
the mutex if it is owned by this thread so that may be problematic. */
lock_ttys::release ();

View File

@ -1347,8 +1347,8 @@ fhandler_pty_master::fhandler_pty_master ()
int
fhandler_pty_master::open (int flags, mode_t)
{
int ntty;
ntty = cygwin_shared->tty.allocate (false);
/* Note that allocate returns with the tty lock set if it was successful. */
int ntty = cygwin_shared->tty.allocate (false);
if (ntty < 0)
return 0;
@ -1361,8 +1361,8 @@ fhandler_pty_master::open (int flags, mode_t)
lock_ttys::release ();
set_flags ((flags & ~O_TEXT) | O_BINARY);
set_open_status ();
//
// FIXME: Do this better someday
/* FIXME: Do this better someday */
fhandler_pty_master *arch = (fhandler_tty_master *) cmalloc_abort (HEAP_ARCHETYPES, sizeof (*this));
*((fhandler_pty_master **) cygheap->fdtab.add_archetype ()) = arch;
archetype = arch;