This is the mail archive of the cygwin-cvs@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[newlib-cygwin] Cleanup in dcrt0.cc and fork.cc


https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;h=81e6c7515da61c8dc8426804f1eeb25981b13d55

commit 81e6c7515da61c8dc8426804f1eeb25981b13d55
Author: Corinna Vinschen <corinna@vinschen.de>
Date:   Thu Dec 3 12:59:56 2015 +0100

    Cleanup in dcrt0.cc and fork.cc
    
            * dcrt0.cc (child_info_fork::alloc_stack): Fix formatting.
            * fork.cc (frok::parent): Fix formatting.
            (child_copy): Change type of res to BOOL.
    
    Signed-off-by: Corinna Vinschen <corinna@vinschen.de>

Diff:
---
 winsup/cygwin/ChangeLog | 6 ++++++
 winsup/cygwin/dcrt0.cc  | 6 +++---
 winsup/cygwin/fork.cc   | 6 +++---
 3 files changed, 12 insertions(+), 6 deletions(-)

diff --git a/winsup/cygwin/ChangeLog b/winsup/cygwin/ChangeLog
index 03cc4e2..149270a 100644
--- a/winsup/cygwin/ChangeLog
+++ b/winsup/cygwin/ChangeLog
@@ -1,5 +1,11 @@
 2015-12-03  Corinna Vinschen  <corinna@vinschen.de>
 
+	* dcrt0.cc (child_info_fork::alloc_stack): Fix formatting.
+	* fork.cc (frok::parent): Fix formatting.
+	(child_copy): Change type of res to BOOL.
+
+2015-12-03  Corinna Vinschen  <corinna@vinschen.de>
+
 	* crt0.cc (dll_crt0_1): Always restore the TEB stack info.  Especially
 	restore DeallocationStack.  Fix comment.
 
diff --git a/winsup/cygwin/dcrt0.cc b/winsup/cygwin/dcrt0.cc
index 9140fa0..a34f643 100644
--- a/winsup/cygwin/dcrt0.cc
+++ b/winsup/cygwin/dcrt0.cc
@@ -472,9 +472,9 @@ child_info_fork::alloc_stack ()
       /* Fork has been called from main thread.  Simply commit the region
 	 of the stack commited in the parent but not yet commited in the
 	 child and create new guardpages. */
-      if (NtCurrentTeb()->Tib.StackLimit > stacklimit)
+      if (NtCurrentTeb ()->Tib.StackLimit > stacklimit)
 	{
-	  SIZE_T commitsize = (PBYTE) NtCurrentTeb()->Tib.StackLimit
+	  SIZE_T commitsize = (PBYTE) NtCurrentTeb ()->Tib.StackLimit
 			      - (PBYTE) stacklimit;
 	  if (!VirtualAlloc (stacklimit, commitsize, MEM_COMMIT, PAGE_READWRITE))
 	    api_fatal ("can't commit child memory for stack %p(%ly), %E",
@@ -484,7 +484,7 @@ child_info_fork::alloc_stack ()
 			     MEM_COMMIT, PAGE_READWRITE | PAGE_GUARD))
 	    api_fatal ("fork: couldn't allocate new stack guard page %p, %E",
 		       guardpage);
-	  NtCurrentTeb()->Tib.StackLimit = stacklimit;
+	  NtCurrentTeb ()->Tib.StackLimit = stacklimit;
 	}
       /* This only affects forked children of a process started from a native
 	 64 bit process, but it doesn't hurt to do it unconditionally.  Fix
diff --git a/winsup/cygwin/fork.cc b/winsup/cygwin/fork.cc
index 1581867..e7b9ea4 100644
--- a/winsup/cygwin/fork.cc
+++ b/winsup/cygwin/fork.cc
@@ -308,7 +308,7 @@ frok::parent (volatile char * volatile stack_here)
   ch.forker_finished = forker_finished;
 
   ch.from_main = &_my_tls == _main_tls;
-  ch.stackbase = NtCurrentTeb()->Tib.StackBase;
+  ch.stackbase = NtCurrentTeb ()->Tib.StackBase;
   ch.stackaddr = NtCurrentTeb ()->DeallocationStack;
   if (!ch.stackaddr)
     {
@@ -316,7 +316,7 @@ frok::parent (volatile char * volatile stack_here)
 	 stack.  If so, the entire stack is committed anyway and StackLimit
 	 points to the allocation address of the stack.  Mark in guardsize that
 	 we must not set up guard pages. */
-      ch.stackaddr = ch.stacklimit = NtCurrentTeb()->Tib.StackLimit;
+      ch.stackaddr = ch.stacklimit = NtCurrentTeb ()->Tib.StackLimit;
       ch.guardsize = (size_t) -1;
     }
   else
@@ -683,7 +683,7 @@ child_copy (HANDLE hp, bool write, ...)
 	  SIZE_T done = 0;
 	  if (here + todo > high)
 	    todo = high - here;
-	  int res;
+	  BOOL res;
 	  if (write)
 	    res = WriteProcessMemory (hp, here, here, todo, &done);
 	  else


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]