reflow comments in wtf

third_party/WebKit/Source/wtf/ThreadingWin.cpp

 /*
  * Copyright (C) 2007, 2008 Apple Inc. All rights reserved.
  * Copyright (C) 2009 Google Inc. All rights reserved.
  * Copyright (C) 2009 Torch Mobile, Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  *
  * 1.  Redistributions of source code must retain the above copyright
@@ skipping 11 matching lines @@
  * DISCLAIMED. IN NO EVENT SHALL APPLE OR ITS CONTRIBUTORS BE LIABLE FOR ANY
  * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
  * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
  * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND
  * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 /*
- * There are numerous academic and practical works on how to implement pthread_cond_wait/pthread_cond_signal/pthread_cond_broadcast
- * functions on Win32. Here is one example: http://www.cs.wustl.edu/~schmidt/win32-cv-1.html which is widely credited as a 'starting point'
- * of modern attempts. There are several more or less proven implementations, one in Boost C++ library (http://www.boost.org) and another
+ * There are numerous academic and practical works on how to implement
+ * pthread_cond_wait/pthread_cond_signal/pthread_cond_broadcast
+ * functions on Win32. Here is one example:
+ * http://www.cs.wustl.edu/~schmidt/win32-cv-1.html which is widely credited as
+ * a 'starting point' of modern attempts. There are several more or less proven
+ * implementations, one in Boost C++ library (http://www.boost.org) and another
  * in pthreads-win32 (http://sourceware.org/pthreads-win32/).
  *
- * The number of articles and discussions is the evidence of significant difficulties in implementing these primitives correctly.
- * The brief search of revisions, ChangeLog entries, discussions in comp.programming.threads and other places clearly documents
- * numerous pitfalls and performance problems the authors had to overcome to arrive to the suitable implementations.
- * Optimally, WebKit would use one of those supported/tested libraries directly. To roll out our own implementation is impractical,
- * if even for the lack of sufficient testing. However, a faithful reproduction of the code from one of the popular supported
- * libraries seems to be a good compromise.
+ * The number of articles and discussions is the evidence of significant
+ * difficulties in implementing these primitives correctly.  The brief search
+ * of revisions, ChangeLog entries, discussions in comp.programming.threads and
+ * other places clearly documents numerous pitfalls and performance problems
+ * the authors had to overcome to arrive to the suitable implementations.
+ * Optimally, WebKit would use one of those supported/tested libraries
+ * directly.  To roll out our own implementation is impractical, if even for
+ * the lack of sufficient testing. However, a faithful reproduction of the code
+ * from one of the popular supported libraries seems to be a good compromise.
  *
- * The early Boost implementation (http://www.boxbackup.org/trac/browser/box/nick/win/lib/win32/boost_1_32_0/libs/thread/src/condition.cpp?rev=30)
- * is identical to pthreads-win32 (http://sourceware.org/cgi-bin/cvsweb.cgi/pthreads/pthread_cond_wait.c?rev=1.10&content-type=text/x-cvsweb-markup&cvsroot=pthreads-win32).
- * Current Boost uses yet another (although seemingly equivalent) algorithm which came from their 'thread rewrite' effort.
+ * The early Boost implementation
+ * (http://www.boxbackup.org/trac/browser/box/nick/win/lib/win32/boost_1_32_0/libs/thread/src/condition.cpp?rev=30)
+ * is identical to pthreads-win32
+ * (http://sourceware.org/cgi-bin/cvsweb.cgi/pthreads/pthread_cond_wait.c?rev=1.10&content-type=text/x-cvsweb-markup&cvsroot=pthreads-win32).
+ * Current Boost uses yet another (although seemingly equivalent) algorithm
+ * which came from their 'thread rewrite' effort.
  *
- * This file includes timedWait/signal/broadcast implementations translated to WebKit coding style from the latest algorithm by
- * Alexander Terekhov and Louis Thomas, as captured here: http://sourceware.org/cgi-bin/cvsweb.cgi/pthreads/pthread_cond_wait.c?rev=1.10&content-type=text/x-cvsweb-markup&cvsroot=pthreads-win32
- * It replaces the implementation of their previous algorithm, also documented in the same source above.
- * The naming and comments are left very close to original to enable easy cross-check.
+ * This file includes timedWait/signal/broadcast implementations translated to
+ * WebKit coding style from the latest algorithm by Alexander Terekhov and
+ * Louis Thomas, as captured here:
+ * http://sourceware.org/cgi-bin/cvsweb.cgi/pthreads/pthread_cond_wait.c?rev=1.10&content-type=text/x-cvsweb-markup&cvsroot=pthreads-win32
+ * It replaces the implementation of their previous algorithm, also documented
+ * in the same source above.  The naming and comments are left very close to
+ * original to enable easy cross-check.
  *
- * The corresponding Pthreads-win32 License is included below, and CONTRIBUTORS file which it refers to is added to
+ * The corresponding Pthreads-win32 License is included below, and CONTRIBUTORS
+ * file which it refers to is added to
  * source directory (as CONTRIBUTORS.pthreads-win32).

[dcheng, 2016/10/01 19:58:17]

Nit: merge into previous line

[Nico, 2016/10/02 00:49:28]

Done.

  */
 
 /*
  *      Pthreads-win32 - POSIX Threads Library for Win32
  *      Copyright(C) 1998 John E. Bossom
  *      Copyright(C) 1999,2005 Pthreads-win32 contributors
  *
  *      Contact Email: rpj@callisto.canberra.edu.au
  *
  *      The current list of contributors is contained
@@ skipping 29 matching lines @@
 #include "wtf/ThreadSpecific.h"
 #include "wtf/ThreadingPrimitives.h"
 #include "wtf/WTFThreadData.h"
 #include "wtf/dtoa/double-conversion.h"
 #include <errno.h>
 #include <process.h>
 #include <windows.h>
 
 namespace WTF {
 
-// THREADNAME_INFO comes from <http://msdn.microsoft.com/en-us/library/xcb2z8hs.aspx>.
+// THREADNAME_INFO comes from
+// <http://msdn.microsoft.com/en-us/library/xcb2z8hs.aspx>.
 #pragma pack(push, 8)
 typedef struct tagTHREADNAME_INFO {
   DWORD dwType;      // must be 0x1000
   LPCSTR szName;     // pointer to name (in user addr space)
   DWORD dwThreadID;  // thread ID (-1=caller thread)
   DWORD dwFlags;     // reserved for future use, must be zero
 } THREADNAME_INFO;
 #pragma pack(pop)
 
 static Mutex* atomicallyInitializedStaticMutex;
 
 void lockAtomicallyInitializedStaticMutex() {
   ASSERT(atomicallyInitializedStaticMutex);
   atomicallyInitializedStaticMutex->lock();
 }
 
 void unlockAtomicallyInitializedStaticMutex() {
   atomicallyInitializedStaticMutex->unlock();
 }
 
 void initializeThreading() {
   // This should only be called once.
   ASSERT(!atomicallyInitializedStaticMutex);
 
-  // StringImpl::empty() does not construct its static string in a threadsafe fashion,
-  // so ensure it has been initialized from here.
+  // StringImpl::empty() does not construct its static string in a threadsafe
+  // fashion, so ensure it has been initialized from here.
   StringImpl::empty();
   StringImpl::empty16Bit();
   atomicallyInitializedStaticMutex = new Mutex;
   wtfThreadData();
   initializeDates();
   // Force initialization of static DoubleToStringConverter converter variable
   // inside EcmaScriptConverter function while we are in single thread mode.
   double_conversion::DoubleToStringConverter::EcmaScriptConverter();
 }
 
@@ skipping 196 matching lines @@
 }
 
 void ThreadCondition::wait(MutexBase& mutex) {
   m_condition.timedWait(mutex.impl(), INFINITE);
 }
 
 bool ThreadCondition::timedWait(MutexBase& mutex, double absoluteTime) {
   DWORD interval = absoluteTimeToWaitTimeoutInterval(absoluteTime);
 
   if (!interval) {
-    // Consider the wait to have timed out, even if our condition has already been signaled, to
-    // match the pthreads implementation.
+    // Consider the wait to have timed out, even if our condition has already
+    // been signaled, to match the pthreads implementation.
     return false;
   }
 
   return m_condition.timedWait(mutex.impl(), interval);
 }
 
 void ThreadCondition::signal() {
   m_condition.signal(false);  // Unblock only 1 thread.
 }
 
 void ThreadCondition::broadcast() {
   m_condition.signal(true);  // Unblock all threads.
 }
 
 DWORD absoluteTimeToWaitTimeoutInterval(double absoluteTime) {
   double currentTime = WTF::currentTime();
 
   // Time is in the past - return immediately.
   if (absoluteTime < currentTime)
     return 0;
 
-  // Time is too far in the future (and would overflow unsigned long) - wait forever.
+  // Time is too far in the future (and would overflow unsigned long) - wait
+  // forever.
   if (absoluteTime - currentTime > static_cast<double>(INT_MAX) / 1000.0)
     return INFINITE;
 
   return static_cast<DWORD>((absoluteTime - currentTime) * 1000.0);
 }
 
 #if ENABLE(ASSERT)
 static bool s_threadCreated = false;
 
 bool isAtomicallyInitializedStaticMutexLockHeld() {
   return atomicallyInitializedStaticMutex &&
          atomicallyInitializedStaticMutex->locked();
 }
 
 bool isBeforeThreadCreated() {
   return !s_threadCreated;
 }
 
 void willCreateThread() {
   s_threadCreated = true;
 }
 #endif
 
 }  // namespace WTF
 
 #endif  // OS(WIN)