Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.10.0
    • Component/s: Build
    • Labels:
      None
    • Environment:
      SGI Irix

      Description

      In file thread.cpp the function Thread::InterlockedIncrement is missing an #else directive:

      -------------------------- from Thread.cpp------------------------------
      #elif defined(HAVE_MS_THREAD)
      #if _MSC_VER == 1200 // MSDEV 6
      return ::InterlockedDecrement((long *)val);
      #else
      return ::InterlockedDecrement(val);
      #endif // _MSC_VER
      #else /****** Without this line the function does not return a value
      return *val - 1; // unsafe
      #endif
      ------------------------------------------------------------------------

      Also: should not the #else case be
      return --(*val) ?
      As it stands now *val is left unchanged...

      Similarly for function Thread::InterlockedDecrement.

        Attachments

          Activity

            People

            • Assignee:
              carnold@apache.org Curt Arnold
              Reporter:
              ingekr Inge K. Eliassen
            • Votes:
              1 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: