Details

    • Type: Task Task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.7
    • Fix Version/s: 0.7
    • Component/s: C++ Client
    • Labels:
    • Environment:

      Mac OS X

      Description

      The first stop in client compilation on Mac OS X is the return value of Thread::id(). It seams that this method should be removed, so lets start from removing the dependencies.

        Issue Links

          Activity

          Hide
          Andrew Stitcher added a comment -

          This issue should now be fixed - although there is improvement possible to Thread::logId()

          Show
          Andrew Stitcher added a comment - This issue should now be fixed - although there is improvement possible to Thread::logId()
          Hide
          Andrew Stitcher added a comment - - edited

          There are 2 major uses of Thread::id():

          1. For use in the logging subsystem to indicate the thread a message has come from (encapsulated in logId()). This could use some other way to indicate the thread - perhaps a sequential thread number starting at 0 - This would make the numbers easier to read too.

          2. To compare Thread identity - it seems to me that this would be better achieved by writing a Thread::operator== (which could use pthread_t under the covers if necessary).

          Show
          Andrew Stitcher added a comment - - edited There are 2 major uses of Thread::id(): 1. For use in the logging subsystem to indicate the thread a message has come from (encapsulated in logId()). This could use some other way to indicate the thread - perhaps a sequential thread number starting at 0 - This would make the numbers easier to read too. 2. To compare Thread identity - it seems to me that this would be better achieved by writing a Thread::operator== (which could use pthread_t under the covers if necessary).

            People

            • Assignee:
              Unassigned
              Reporter:
              Bruno Matos
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development