« Previous | Next » 

Revision 96cc8d6f

ID96cc8d6f2ab6561fc9a067fd3fe0a5e0f4abb24e

Added by Simon Tenbusch about 8 years ago

[nullduration] fixed the Bugs, Daniel mailed:
-- (hoch) INSTALL behauptet NOBARRIER wäre default. in meinem checkout
ist dies nicht wahr.

-- (unwichtig) include/cthreadpool.h beginnt mit einer forward
declaration von cthreadpool - direkt vor der definition

-- (mittel) cspinningthreadpool.cc hat file-level comparision
operatoren auf cSpinningThreadPool::threadState_t ist, was wiederum
ein typedef auf cSpinningThreadPool::threadState_t ist (C-Style struct
typedef, in C++ einfach unfug, da kein extra struct namespace
vorhanden). dieses struct ist jedoch public, so dass die comparision
operatoren auf einen öffentlichen typen nur in einem translation unit
vorhanden sind! Ebenfalls c-style ist der returntyp der operatoren:
int (es ist ein bool-typ gemeint, also warum nicht bool?)

(mittel) die pause instruktion in beiden lock schleifen ist nicht
volatile, siehe hier für erklärung:
http://www.ibiblio.org/gferg/ldp/GCC-Inline-Assembly-HOWTO.html#ss5.4

Files

  • added
  • modified
  • copied
  • renamed
  • deleted

View differences